Skip to content

Deletion#4

Open
Woojgh wants to merge 14 commits into
masterfrom
deletion
Open

Deletion#4
Woojgh wants to merge 14 commits into
masterfrom
deletion

Conversation

@Woojgh
Copy link
Copy Markdown
Owner

@Woojgh Woojgh commented Jul 2, 2017

No description provided.

Comment thread 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

Comment thread src/BST.py
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

Comment thread src/BST.py
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

Comment thread src/BST.py
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