Remove deprecated add_axioms_for_concat_code_point method#8924
Remove deprecated add_axioms_for_concat_code_point method#8924tautschnig wants to merge 1 commit intodiffblue:developfrom
Conversation
Inline the body of the deprecated add_axioms_for_concat_code_point method at its sole call site in add_axioms_for_function_application. This removes the method declaration from the header and the separate method definition while preserving the functionality needed by JBMC's StringBuilder.appendCodePoint and StringBuffer.appendCodePoint. Fixes: diffblue#8014 Co-authored-by: Kiro <kiro-agent@users.noreply.github.com>
3f50573 to
c3d113d
Compare
There was a problem hiding this comment.
Pull request overview
This PR removes the deprecated string_constraint_generatort::add_axioms_for_concat_code_point helper by inlining its implementation at its only call site inside add_axioms_for_function_application, preserving behavior used by JBMC’s StringBuilder.appendCodePoint / StringBuffer.appendCodePoint handling (Fixes #8014).
Changes:
- Removed the deprecated
add_axioms_for_concat_code_pointdeclaration from the generator header. - Inlined the former implementation directly into the
ID_cprover_string_concat_code_point_funcdispatch branch. - Deleted the obsolete definition from
string_concatenation_builtin_function.cpp.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
src/solvers/strings/string_constraint_generator.h |
Drops the deprecated method declaration from the public API surface of the generator class. |
src/solvers/strings/string_constraint_generator_main.cpp |
Inlines the concat-code-point axiom generation logic into the function-application dispatcher. |
src/solvers/strings/string_concatenation_builtin_function.cpp |
Removes the deprecated method definition that is no longer referenced. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #8924 +/- ##
===========================================
- Coverage 80.46% 80.45% -0.01%
===========================================
Files 1704 1704
Lines 188665 188665
Branches 73 73
===========================================
- Hits 151805 151799 -6
- Misses 36860 36866 +6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Inline the body of the deprecated add_axioms_for_concat_code_point method at its sole call site in add_axioms_for_function_application. This removes the method declaration from the header and the separate method definition while preserving the functionality needed by JBMC's StringBuilder.appendCodePoint and StringBuffer.appendCodePoint.
Fixes: #8014
Co-authored-by: Kiro kiro-agent@users.noreply.github.com