taavi/xdebug — comment on commit cae8255ea43b9bc2087256968d20e221fa0863fb
File: xdebug_code_coverage.c line 335
By taavi:

It changes the data stored, but not what the outside world sees (assuming I understand this file properly).

As far as I can tell, the only way that this coverage data gets out is via add_lines() (formerly xdebug_hash_apply(…, add_line)).

Old line 625 returned "1" if the line got coverage, never the number of times it was hit. Its equivalent (on new line 634) preserves that behaviour. Hence, I don't think any callers will notice, and it means that line->count fits in 1 bit instead of requiring something on the order of 32bits. :)

Does that logic hold?