Allow user to use -debug=1 to enable all debugging #6164

pull lpescher wants to merge 3 commits into bitcoin:master from lpescher:UpdateHelpMessage#4219 changing 3 files +3 −2

lpescher commented at 4:55 AM on May 20, 2015: contributor

This is more consistent with other command line arguments and more intuitive. Originally, a user would specify "-debug=" to enable all debugging; which still works. The documentation and --help pages are also updated

Addresses Issue #4219

Make command line option to show all debugging consistent with similar options

Most people expect a value of 1 to enable all for command line arguments.
However to do this for the -debug option you must type "-debug=".
This has been changed to allow "-debug=1" as well as "-debug=" to
enable all debug logging

cb87386241

Update documentation to match the #4219 change f60bb5ee6d
Update help message to match the #4219 change aa41bc84b1
jonasschnelli commented at 9:20 AM on May 20, 2015: contributor

tested ACK.

I think this make sense. -debug=1 should act the same as -debug.
laanwj commented at 10:48 AM on May 20, 2015: member

utACK - I'd prefer if you squash this into one commit as all three changes form one atomic whole.
laanwj added the label Improvement on May 20, 2015
fanquake commented at 11:21 AM on May 20, 2015: member

ACK after squashing

On Wednesday, May 20, 2015, Wladimir J. van der Laan < notifications@github.com> wrote:

utACK - I'd prefer if you squash this into one commit as all three changes form one atomic whole.

Reply to this email directly or view it on GitHub #6164 (comment).
laanwj merged this on Jun 1, 2015
laanwj closed this on Jun 1, 2015
laanwj referenced this in commit 8d05ec7bda on Jun 1, 2015
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels

Linked (view graph)

#4219 -debug=1 should show all debug messages