-
Notifications
You must be signed in to change notification settings - Fork 591
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Split cxxopts.hpp into multiple files #338
Comments
…ostly one for each class.
…ostly one for each class.
I always wanted this to be single header to just drop into a project, and it still doesn't seem huge yet. If there was a nice way to generate a single header then I think it could be broken up. |
@jarro2783 : Why is it important to be able to drop a single file rather than a subfolder? Anyway, to create a single header, you run:
and you get quom with:
|
…ostly one for each class.
…ostly one for each class.
…ostly one for each class.
Yeah I guess having multiple files is not so bad. I'll look at this after some of the other changes because it will mess up PRs. |
So, yeah, the different PRs tend to clash. When you decide which of them you like, I'll rebase them onto each other so that you can merge. |
One can observe a decomposition of
cxxopts.hpp
into several parts:And the above is also the order of dependencies.
Or, actually, maybe we could take this another step and have one file per (non-trivial) class, so separate files for Option, Option Adder, Options, OptionParser etc.
So, how about we split the huge single file into a file for each of these? With
cxxopts.hpp
remaining as an "includer" of all the others?This will:
What do you think, @jarro2783 ?
The text was updated successfully, but these errors were encountered: