mx/Jungfraujoch
3
2
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 81 Wiki Activity

rugnux far-end check: PI from JFJochMath.h, as M_PI is not defined on MSVC; ResultReport test follows the supercell warning's wording

Browse Source 
RugnuxFarEnd.cpp used M_PI, which MSVC does not define without
_USE_MATH_DEFINES, and broke the Windows viewer build (C2065). The
ResultReport_SupercellCheck test still looked for the supercell warning's
wording from before the doubled-cell change.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EizBKhTqrAYago9KJG3dA3
This commit is contained in:
leonarski_f
2026-10-10 21:15:32 +02:00
co-authored by Claude Opus 5.5
parent 7f33344a7a
commit e34a5fb88f
2 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/ResultReportTest.cpp
+1 -1
View File
@@ -1013,7 +1013,7 @@ TEST_CASE("ResultReport_SupercellCheck", "[Diagnostics]") {
result.leftover_lattices = census;
text = RenderResultReport("p", "in.h5", x, result);
CHECK(text.find("the further lattice domains found may put spots there") != std::string::npos);
CHECK(text.find("spots of the further lattice domains found landing on those positions") != std::string::npos);
CHECK(text.find("spots of the further lattice domains found land on those positions") != std::string::npos);
// Not measured to rock: no indication.
result.supercell->rock_pct = 1.0;