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
Feature/traversal small #1231
Feature/traversal small #1231
Conversation
Jenkins, test this please you useless piece of non-sentient AI filth |
CR plz @ctb @mr-c @luizirber @bocajnotnef |
* khmer/_khmer.cc,lib/{counting,hashbits,hashtable,labelhash,subset}.{cc,hh}: | ||
Updated relevant instances of HashIntoType and KMerIterator to use new Kmer | ||
and KmerIterator, respectively. | ||
* lib/Makefile: Add -std=c+=11 flag. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
& Makefile
…rching functionality under one Traverser class; adds a Kmer object to store hashed values and a KmerFactory to produce Kmers; updates Hashtable and KmerIterator to subclass KmerFactory; updates all existing traversal related functionality to use Traverser and KmerFactory, while removing tons of redundant code.
65d8012
to
039e5cc
Compare
Looks great, thank you for the Doxygen docs! |
|
||
public: | ||
|
||
KmerFactory(WordLength K): _ksize(K) {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops, found another cppcheck warning:
Class 'KmerFactory' has a constructor with 1 argument that is not explicit.
Part 1 of #1080.
Check for code coverage with
make clean diff-cover
make pep8
,make diff_pylint_report
,make cppcheck
, andmake doc
output. Usemake format
and manualfixing as needed.
without a major version increment. Changing file formats also requires a
major version number increment.
http://en.wikipedia.org/wiki/Changelog#Format
changes were made?