Skip to content

Conversation

@Matthew-Whitlock
Copy link
Collaborator

Done to match the typical standard for namespace capitalization, but also to avoid a specific name clash nuisance in the Fenix Lammps work (which has a Fenix class, to follow their naming conventions).

fenix->fenix_rt and Fenix::->fenix::

Explaining 66 more deletions than insertions:
61: Deleted globals.cpp by declaring fenix_rt as inline
1: Remove globals.cpp from src/CMakeLists.txt
2: Remove some excess whitespace from fenix_ext.hpp
2: Remove whitespace and debug print from fenix_opt.cpp

Otherwise insertions/deletions are balanced as expected.

@Matthew-Whitlock Matthew-Whitlock merged commit dc00442 into develop Nov 11, 2025
7 of 10 checks passed
@Matthew-Whitlock Matthew-Whitlock deleted the namespace-refactor branch November 11, 2025 20:52
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.

2 participants