Skip to content

Conversation

@hoche
Copy link
Owner

@hoche hoche commented Dec 16, 2025

Note

Introduces a dedicated command-line parser, refactors main.cpp to use it with validation, and adds extensive unit tests with build integration.

  • CLI Parsing:
    • New src/command_line_parser.{h,cpp} implementing option parsing, help output, and ValidateCommandLine (enforces -maxpages constraints and presence of TX sites).
    • src/main.cpp refactored to use CommandLineOptions and ParseCommandLine/ValidateCommandLine; replaces ad-hoc parsing and wires parsed filenames through execution paths.
  • Tests:
    • Add tests/command_line_parser_test.cpp with broad coverage of flags, clamping, conflicts, and validation.
  • Build System:
    • Include src/command_line_parser.cpp in CMakeLists.txt and test sources; update references in src/CMakeLists.txt.

Written by Cursor Bugbot for commit 0286a07. This will update automatically on new commits. Configure here.

@hoche hoche merged commit ac51ee1 into main Dec 16, 2025
18 checks passed
@hoche hoche deleted the feature/add_command_line_switch_tests branch December 16, 2025 23:58
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