Remove ycm extra configuration files
forgotten case
Merge pull request #312 from broadinstitute/ak_fix_samples_to_include_in_variant_reader

fix for forgotten list of samples to include
Merge pull request #309 from broadinstitute/mc_remove_ycm_files

Remove ycm extra configuration files
Read group class

Read group class

Minor changes base don code review

Minor improvements in code readabiity
Merge pull request #294 from broadinstitute/db_read_groups

ReadGroup class along with some support in SamHeader.
1. Modified SharedFieldIterator and IndividualFieldValueIterator to
handle bcf_*_vector_end values. This is done by passing the end value of
the pointer in the constructor and modifying the "==" operator of the
iterator. The idea is that 2 iterators x,y are equal iff
 (a) x and y are identical  OR
 (b) (x == end && *y == bcf_*_vector_end) || (y == end && *x == bcf_*_vector_end)
2. Inserted test case fields for variable length vectors
Modified behavior of SharedFieldIterator and
IndividualFieldValueIterator so that:
(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
After rebase against master
(a) Cache current pointee value in SharedIterator and
IndividualFieldValueIterator to reduce overhead
(b) Moved common code to utils
Incorporated David's suggestions:
a. Use constructor with full parameter list in iterators
b. Modified operator[]
c. Moved variable length field tests to different function
Changes after rebase
Operator[] in SharedFieldIterator, IndividualFieldIterator and IndividualFieldValueIterator works relative to m_current_data_ptr now