Skip to content
This repository was archived by the owner on Dec 16, 2022. It is now read-only.

Conversation

@kgururaj
Copy link
Collaborator

DO NOT MERGE - only for testing Travis build

jmthibault79 and others added 9 commits October 2, 2014 16:31
create a new *production* mode to eliminate the -finline-functions that
is no longer supported by clang 3.5

also create a global fixture for initialization of global variables in
the variant reader test.

closes #290
Vector end values were not passing through to the caller
when accessing alleles via operator[] in the Genotype class.
Read group class

Minor changes base don code review

Minor improvements in code readabiity
handle bcf_*_vector_end values. This is done by passing the end value of
the pointer in the constructor and modifying the increment operators.

(a) Operators ++ and += advance to the end of the vector if the value at
the new iterator location is equal to bcf_*_vector_end.
(b) Operator == is same as the original
(c) Operators -- and -= unchanged

2. Inserted test cases for variable length vectors

3. Cache current pointee value in SharedIterator and
IndividualFieldValueIterator to reduce overhead

4. Use constructor with full parameter list in iterators

5. Operator[] in SharedFieldIterator, IndividualFieldIterator and IndividualFieldValueIterator works relative to m_current_data_ptr now

6. Added tests for operator[] for SharedFieldIterator, IndividualFieldIterator and IndividualFieldValueIterator
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) when pulling ac96849 on intel_test_branch into 7bdce8e on master.

@kgururaj kgururaj closed this Oct 21, 2014
@kgururaj kgururaj deleted the intel_test_branch branch October 21, 2014 16:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants