Skip to content

Iterator interface for picks, and minor code cleanup. #1

Merged
sdleffler merged 6 commits intosdleffler:masterfrom
jswrenn:master
Mar 15, 2017
Merged

Iterator interface for picks, and minor code cleanup. #1
sdleffler merged 6 commits intosdleffler:masterfrom
jswrenn:master

Conversation

@jswrenn
Copy link
Contributor

@jswrenn jswrenn commented Mar 15, 2017

This pull requests implements IntoIterator for &AliasTable, and introduces AliasTableIterator (which implements Iterator).

A few minor code-smells are corrected, too:

  • replaced two multi-line routines with more succinct calls to standard-library functions (namely extend)
  • replaced the num dependency with num_traits, the only subset of num that was used
  • made clippy happy

I use the pick iterator here.

@sdleffler
Copy link
Owner

Nice work! Thanks for the PR.

@sdleffler sdleffler merged commit 32711cc into sdleffler:master Mar 15, 2017
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.

2 participants