mgrettondann/partridge-cpp
1
0
Fork 0
Code Issues 4 Pull Requests Activity

results: fix Debug rendering assertion #18

Merged
mcp merged 2 commits from codex/issue-7-results-assert into main 2026-07-30 16:47:26 +01:00
Conversation 0 Commits 2 Files Changed 4
+5 -4
2 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ebfc80546a - Show all commits
TESTING.md
+3 -3
View File
@@ -38,6 +38,6 @@ ctest --test-dir build-ubsan --output-on-failure
``` ```
The sanitizer flags shown are supported by Clang and GCC. Other compilers may The sanitizer flags shown are supported by Clang and GCC. Other compilers may
require different flags. Debug compilation and a feasible solver run currently require different flags. A feasible solver run currently exposes the
expose the pre-existing defects tracked by issues #7 and #14 respectively; the pre-existing defect tracked by issue #14; the test additions deliberately do
test additions deliberately do not include fixes for those separate issues. not include its separate fix.
main.cc
+2 -1
View File
@@ -85,7 +85,8 @@ namespace {
auto set(std::string &s, size_t x, size_t y, char c) const noexcept -> void { auto set(std::string &s, size_t x, size_t y, char c) const noexcept -> void {
assert(x < length_); assert(x < length_);
assert(y < length_); assert(y < length_);
assert(grid_[x + y * length_] != c); // Size labels may replace interior spaces, but must not duplicate a write.
assert(s[x + y * length_] != c);
s[x + y * length_] = c; s[x + y * length_] = c;
} }