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
Normalise config descriptions and add default values to most of them #2221
Conversation
36a7f75
to
a866643
Compare
a866643
to
376a1df
Compare
I've wrapped every single config description to 79 chars as you suggested @FeralChild64, plus I've made numerous other improvements. It's about 90% there now, and I could keep doing this for another day, but I'm already feeling dizzy, so I'll stop at this point 😄 |
36e5959
to
b3c8ba3
Compare
b3c8ba3
to
bc5639a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Conf files are looking very consistent and less ragged - very nice cleanup, @johnnovak!
0f6e91a
to
4db5ead
Compare
Even though KiB and MiB are the proper base-2 units as defined by IEC, they have never caught on, and Microsoft unilaterally used MB and KB to refer to memory in base-2 units during the DOS era.
4db5ead
to
5a60fea
Compare
The big improvement here is listing the default values for most config settings. Some of the more elaborate ones need the
sprintf
treatment; I left those for another pass. Anyway, this takes care of the low-hanging fruit, which is about 90% of all config settings.Almost started re-wrapping the descriptions because some lines got longer after adding the defaults, but quickly realised it's pointless because in the end it's gonna be all over the place anyway (because of the varying levels of indentation, depending on the length of the param names).