jarro2783 / cxxopts

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

Ordered vector of group names to preserve order of groups #416

Closed nigels-com closed 10 months ago

nigels-com commented 10 months ago

Thanks for cxxopts, it's fits nicely with our needs.

I experimented with the grouping support today. I noticed that the order of the groups isn't insertion order - it's based on the std::map comparator. So I felt that we could add a std::vector of the group names here in the order of insertion. Two benefits:

For consideration.


This change is Reviewable

jarro2783 commented 10 months ago

Thanks, looks good.