Skip to content

fix(BRepRayTracing): use shared_mutex#1292

Merged
BotellaA merged 4 commits into
nextfrom
fix/tracing-shared
Jun 22, 2026
Merged

fix(BRepRayTracing): use shared_mutex#1292
BotellaA merged 4 commits into
nextfrom
fix/tracing-shared

Conversation

@BotellaA

Copy link
Copy Markdown
Member

No description provided.

@BotellaA BotellaA requested review from MoulardM and panquez June 22, 2026 09:17
@github-actions

github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v20.1.8) reports: 2 concern(s)
  • src/geode/model/helpers/ray_tracing.cpp:46:64: warning: [readability-identifier-naming]

    invalid case style for function 'directions_2D'

       46 |     const std::array< geode::Vector2D, NUMBER_2D_DIRECTIONS >& directions_2D()
          |                                                                ^~~~~~~~~~~~~
          |                                                                directions_2_d
  • src/geode/model/helpers/ray_tracing.cpp:64:64: warning: [readability-identifier-naming]

    invalid case style for function 'directions_3D'

       64 |     const std::array< geode::Vector3D, NUMBER_3D_DIRECTIONS >& directions_3D()
          |                                                                ^~~~~~~~~~~~~
          |                                                                directions_3_d

Have any feedback or feature suggestions? Share it here.

@BotellaA BotellaA merged commit 7159019 into next Jun 22, 2026
20 checks passed
@BotellaA BotellaA deleted the fix/tracing-shared branch June 22, 2026 19:02
@BotellaA

Copy link
Copy Markdown
Member Author

🎉 This PR is included in version 17.3.3-rc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@BotellaA

Copy link
Copy Markdown
Member Author

🎉 This PR is included in version 17.3.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants