brbzull0 opened a new pull request, #12934:
URL: https://github.com/apache/trafficserver/pull/12934

   ArgParser: apply default values after dependency validation
   `validate_dependencies()` incorrectly triggers on options that have default 
values but were not explicitly passed by the user.
   
   The root cause is that `append_option_data()` populates default values into 
the Arguments map before `validate_dependencies()` runs. When 
`validate_dependencies()` calls `ret.get(key)` for an option with a default, 
the lookup finds the entry and sets `_is_called = true`, making the option 
appear "used" even though the user never specified it on the command line.
   
   Fix by extracting the default-value loop into `apply_option_defaults()` and 
calling it after `validate_dependencies()` in `parse()`.
   
   Test also checks for `Case sensitive short options` which I needed to make 
sure they are working.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to