Skip to content

Commit

Permalink
soci bug fix: get_affected_rows was broken when re-using prepared sta…
Browse files Browse the repository at this point in the history 
…tements
  • Loading branch information
@MonsieurNicolas
MonsieurNicolas committed Oct 21, 2015
1 parent e8627ac commit 54c018b
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions lib/soci/src/backends/sqlite3/statement.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -193,6 +193,7 @@ sqlite3_statement_backend::bind_and_execute(int number)
statement_backend::exec_fetch_result retVal = ef_no_data;

long long rowsAffectedBulkTemp = 0;
rowsAffectedBulk_ = -1;

int const rows = static_cast<int>(useData_.size());
for (int row = 0; row < rows; ++row)
Expand Down

5 comments on commit 54c018b

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from jedmccaleb
at MonsieurNicolas@54c018b

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging MonsieurNicolas/stellar-core/sociFix = 54c018b into auto

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MonsieurNicolas/stellar-core/sociFix = 54c018b merged ok, testing candidate = 0a50758

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/86699113

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 0a50758

Please sign in to comment.