Skip to content

fix: replace bare std::exception throws with descriptive place holder…#12

Open
e-hatch wants to merge 8 commits into
yhirose:masterfrom
e-hatch:master
Open

fix: replace bare std::exception throws with descriptive place holder…#12
e-hatch wants to merge 8 commits into
yhirose:masterfrom
e-hatch:master

Conversation

@e-hatch
Copy link
Copy Markdown

@e-hatch e-hatch commented Apr 29, 2026

replace bare std::exception throws with placeholders

Changes

  • verify() now throws runtime_error with the SQLite error string via sqlite3_errstr(rc)
  • Iterator::operator++() throws runtime_error via sqlite3_errmsg(sqlite3_db_handle(stmt_)) on unexpected step
  • Iterator::operator++() also throws std::logic_error on null statement handle

@e-hatch e-hatch force-pushed the master branch 7 times, most recently from 61c52f9 to 80ef91f Compare May 1, 2026 12:38
@e-hatch e-hatch force-pushed the master branch 3 times, most recently from 1181700 to a5c4a5d Compare May 5, 2026 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant