Alan-Jowett / bdd_test / 19213796942

88%

push

github

Refactor: Move implementation details into anonymous namespaces (#21)

Improve code organization and namespace hygiene by moving internal
helper functions into anonymous namespaces while keeping exported
functions in the global namespace.

Changes by file:
- src/main.cpp: Move write_bdd_nodes_to_stream, get_output_path,
  and convert_to_bdd into anonymous namespace
- src/expression_parser.cpp: Move trim and parse_expression into
  anonymous namespace
- src/bdd_graph.cpp: Move all BDD DOT generation helper functions
  into anonymous namespace

Benefits:
- Better encapsulation of implementation details
- Prevents symbol conflicts between translation units
- Clearer separation of public API vs internal helpers
- Follows C++ best practices for namespace organization
- No functional changes - all tests pass

Only main() and functions declared in headers remain in global namespace.

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Alan Jowett <alan.jowett@microsoft.com>

647 of 763 branches covered (84.8%)

Branch coverage included in aggregate %.

601 of 645 relevant lines covered (93.18%)

221.5 hits per line