From 2db9d0ecd19defe3a9e70fd297f77031779b386b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?S=C3=A9bastien=20Rombauts?= Date: Wed, 23 Sep 2026 11:53:06 +0200 Subject: [PATCH] Remove deprecated Savepoint::rollback() --- CHANGELOG.md | 4 ++++ include/SQLiteCpp/Savepoint.h | 2 -- tests/Savepoint_test.cpp | 2 +- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 433f18d5..4a06cd25 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -16,6 +16,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 - Update googletest to v1.18.0 (#583) - Improve tests for `Savepoint::rollbackTo()` and scope-exit rollback semantics (#584) +### Removed + +- Remove the deprecated `Savepoint::rollback()` alias; use `rollbackTo()` instead (#585) + ## [3.4.0] - 2026-09-21 ### Added diff --git a/include/SQLiteCpp/Savepoint.h b/include/SQLiteCpp/Savepoint.h index 31fe2b6e..a98b8d94 100644 --- a/include/SQLiteCpp/Savepoint.h +++ b/include/SQLiteCpp/Savepoint.h @@ -86,8 +86,6 @@ class SQLITECPP_API Savepoint * @brief Rollback to the savepoint, but don't release it. */ void rollbackTo(); - // @deprecated same as rollbackTo(); - void rollback() { rollbackTo(); } private: Database& mDatabase; ///< Reference to the SQLite Database Connection diff --git a/tests/Savepoint_test.cpp b/tests/Savepoint_test.cpp index c55004d6..6b095039 100644 --- a/tests/Savepoint_test.cpp +++ b/tests/Savepoint_test.cpp @@ -40,7 +40,7 @@ TEST(Savepoint, releaseAndRollback) // Releasing or rolling back an already released savepoint throws. EXPECT_THROW(savepoint.release(), SQLite::Exception); - EXPECT_THROW(savepoint.rollback(), SQLite::Exception); + EXPECT_THROW(savepoint.rollbackTo(), SQLite::Exception); } // Automatic rollback if release() is not called before the end of scope.