Skip to content

OCCT dual-version harden: Handle aliases, failure messages, warning cleanup#197

Merged
trailcode merged 7 commits into
mainfrom
Trailcode/gen-fixes-refactor
Jul 12, 2026
Merged

OCCT dual-version harden: Handle aliases, failure messages, warning cleanup#197
trailcode merged 7 commits into
mainfrom
Trailcode/gen-fixes-refactor

Conversation

@trailcode

@trailcode trailcode commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • Harden shared src/ for desktop OCCT 8 vs WASM OCCT 7.9.3: standard_failure_message(), *_ptr aliases instead of Handle(T), agent conventions for both.
  • Clear native/WASM compiler warnings (nodiscard clear_tmps, Standard_Real, doc-link wchar narrowing, ImGui dock enum OR under emscripten).

Closes #196

Test plan

  • Native Debug rebuild EzyCad_lib / EzyCad_tests (listed C4834 / C4996 / C4244 gone)
  • WASM build-em-793 ninja link succeeds; dock enum warning addressed
  • CI Windows MSVC green
  • Spot-check fillet/chamfer/revolve error paths still surface OCCT messages
  • Spot-check Esc cancel still two-stage (tmp tool then parent mode)

Notes

  • Temporary agent note agents/conventions/occt-wasm-dual-version.md should be retired when wasm on OCCT 8 is supported.
  • Prefer agents/conventions/occt-handles.md for new handle types (add to utl_types.h).

Made with Cursor

@trailcode trailcode merged commit b762eca into main Jul 12, 2026
3 checks passed
@trailcode trailcode deleted the Trailcode/gen-fixes-refactor branch July 12, 2026 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OCCT dual-version (desktop 8 / WASM 7.9.3): Handle aliases, failure messages, build warning cleanup

1 participant