As the design of the command line class was being developed, I realized that the Tester class should not be looping through all of the command line arguments. This lead to the problem mentioned at the end of the last post when other options are specified. The current version and test options are mutually exclusive - only one should be specified.
The version option was already checking to be make it is the only option specified. The Tester class was modified to also make sure only one of the test options are specified - there is no reason to loop through the command line arguments since the number of arguments must match the form used (either one of two arguments). This greatly simplified the Tester code.
All the access read only Tester functions were made constant (previously missed). A new access function was also added to return a list of valid test options, which is used by the caller to construct the usage message.
[commit 710414e4ba]
Subscribe to:
Post Comments (Atom)
No comments:
Post a Comment
All comments and feedback welcomed, whether positive or negative.
(Anonymous comments are allowed, but comments with URL links or unrelated comments will be removed.)