Smattr / clink

a modern re-implementation of Cscope
The Unlicense
46 stars 2 forks source link

fix: do not ignore record insertion failure during DB construction #245

Closed Smattr closed 5 months ago

Smattr commented 6 months ago

319369a9c7c650bc66877c7d9710876bf74fccfe introduced a variable (id) to the record addition that was then threaded through later functions without adjusting the clink_db_add_record call to account for the fact that using this when the call has failed is invalid.

Github: fixes #242 “assertion id >= 0 failed” Reported-by: Timothy Madden