search

jarro2783 / cxxopts

Lightweight C++ command line option parser
MIT License
4.25k stars 590 forks source link

add . as valid option names #358

Closed ryanleary closed 2 years ago

ryanleary commented 2 years ago

This simply allows '.' to exist in arguments. Some libraries use . to denote nested configurations which is not currently possible with cxxopts.

This change is Reviewable

jarro2783 commented 2 years ago

Can you add a test case for this?

ryanleary commented 2 years ago

Test added, thanks!

jarro2783 commented 2 years ago

Looks good, can you rebase?

ryanleary commented 2 years ago

squashed & rebased on latest master