Skip to content

chore: replace #pragma once with old-style include guards#8

Open
mtao wants to merge 1 commit intomainfrom
chore/include-guards
Open

chore: replace #pragma once with old-style include guards#8
mtao wants to merge 1 commit intomainfrom
chore/include-guards

Conversation

@mtao
Copy link
Copy Markdown
Owner

@mtao mtao commented Mar 29, 2026

Summary

  • Convert 8 remaining headers from #pragma once to #if !defined(GUARD) / #define GUARD / #endif style
  • Guard names follow BALSA_PATH_COMPONENTS_HPP convention, derived from the header path relative to each sub-library include root (core, geometry, visualization)
  • Note: build has a pre-existing fmt subproject compatibility issue with GCC 15 unrelated to this change

Convert 8 remaining headers from #pragma once to #if !defined(GUARD)
/ #define GUARD / #endif style. Guard names follow the convention
BALSA_PATH_COMPONENTS_HPP, derived from the header path relative to
each sub-library include root (core, geometry, visualization).
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.

1 participant