Skip to content

Adding the Missing Rust Tree Traversal Functions #326

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Aug 4, 2018
Merged

Adding the Missing Rust Tree Traversal Functions #326

merged 5 commits into from
Aug 4, 2018

Conversation

Wesley-Arrington
Copy link
Contributor

Figured I could quickly add in the dfs_recursive_postorder function in Rust since it is so similar to the dfs_recursive.

The inorder binary tree function is still missing.

- Added Recursive Postorder function
- Modified .md accordingly
@Gathros Gathros added the Implementation Edit This provides an edit to an algorithm implementation. (Code and maybe md files are edited.) label Aug 3, 2018
@Gathros
Copy link
Contributor

Gathros commented Aug 3, 2018

Looks good. Can you add dfs_recursive_inorder_btree too.

@Wesley-Arrington
Copy link
Contributor Author

I don't really know Rust. I figured I could just add in dfs_recursive_postorder because it is just swapping the order of dfs_recursive.

Since I don't really know Rust I don't feel super comfortable doing dfs_recursive_inorder_btree. I might in the future figure out Rust and do dfs_recursive_inorder_btree but I'm not sure when I'll have the time.

@Wesley-Arrington Wesley-Arrington changed the title Adding One of the Missing Rust Tree Traversal Functions Adding the Missing Rust Tree Traversal Functions Aug 4, 2018
@Gathros Gathros merged commit f19f7f2 into algorithm-archivists:master Aug 4, 2018
@Wesley-Arrington Wesley-Arrington deleted the addingRustFnsTree branch August 4, 2018 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Implementation Edit This provides an edit to an algorithm implementation. (Code and maybe md files are edited.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants