2
93%
master: %

Ran 21 Oct 2014 04:44PM UTC

Files 85

Run time 40s

Badge

Embed ▾

Committed 21 Oct 2014 04:32PM UTC coverage: 93.324% (+0.5%) from 92.824%

Job # 1190.2

Build Type

push

travis-ci

Committed by Karthik Gururaj

Commit Message

Modified operator == in IndividualFieldValue to use iterators which
take care of bcf_*_vector_end values implicitly
A more serious bug in the previous implementation of the == operator was
in the way string fields were handled. Points to remember for string
fields:
1. The size() function returns the number of characters allocated for the string.
2. The variable m_num_bytes also stores the number of characters
allocated for the string (same as size())
3. When operator[i] is called for string fields, the address that is
accessed is m_format_ptr->p + i*m_num_bytes (not m_format_ptr->p + i).
The address accessed is outside the original allocated memory.

Moral of the story: use iterators always, or remember the intricacies of
string v/s other types of fields.

Run Details

1384 of 1483 relevant lines covered (93.32%)

925.64 hits per line

broadinstitute / gamgee / 1190 / 2
93%
master: %

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1190.2

broadinstitute / gamgee / 1190 / 2 93% master: %

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1190.2

broadinstitute / gamgee / 1190 / 2
93%
master: %

README BADGES
x