SRombauts / SQLiteCpp / 28404751507

100%

master: 100%

Build Type

Pull #552

github

Committed by SRombauts

Commit Message

Return empty string instead of throwing on null expanded SQL

Keeps the null guard that fixes the UB, but drops the explicit throw. The throw left two lines uncovered (the throw itself and the function's exception-unwind landing brace), neither reachable from a unit test, which tripped the Coveralls no-decrease gate. The empty-string fallback is the alternative noted in the review and adds no new lines.

Pull Request Pull Request #552: Guard null C pointers in Exception, Database, and Statement against UB

Coverage Stats

4 of 4 new or added lines in 1 file covered. (100.0%)

2 existing lines in 2 files now uncovered.

683 of 685 relevant lines covered (99.71%)

30.48 hits per line