Skip to content

Deletion#4

Open
Woojgh wants to merge 14 commits intomasterfrom
deletion
Open

Deletion#4
Woojgh wants to merge 14 commits intomasterfrom
deletion

Conversation

@Woojgh
Copy link
Copy Markdown
Owner

@Woojgh Woojgh commented Jul 2, 2017

No description provided.

setup.py Outdated
name='data-structures',
desctription='A variety of data structures built with Python.',
version='0.1',
author='James Salamonsen',
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should get both authors in here

curr.left = Node(entry)
self.size += 1
Node.parent = curr
curr.left.parent = curr
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment's for search below, the continues aren't necessary

yield node_data

def deletion():
def check_that_balance(self, target=None):
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about these method names lol

curr = curr.right
return curr

def deletion(self, target):
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'delete' is better than 'deletion'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants