Bitcoin-Qt: problem in ParseParameters() when supplying Qt style #2458

issue Diapolo opened this issue on April 4, 2013
  1. Diapolo commented at 12:58 PM on April 4, 2013: none

    I tried to supply -style fusion (Qt5 style) as first command-line parameter to bitcoin-qt.exe, which leads to the client is not further parsing command-line parameters.

    It seems this is causing the problem: https://github.com/bitcoin/bitcoin/blob/master/src/util.cpp#L544

    Workaround is to just supply the -style fusion at the end, but IMHO silently ignoring stuff the parser doesn't like is not the right thing to do!?

  2. laanwj commented at 2:11 PM on April 4, 2013: member

    Isn't there already an issue about this (by me)? Not specifically about qt5 but silently ignoring unknown command line options.

    A possible solution would be to use a command line option parsing library with proper error reporting instead of the hand-rolled stuff we have now. Does boost have one?

    That will be quite a difficult transition though especially as we want to keep backward compatibility.

  3. gavinandresen commented at 2:53 PM on April 4, 2013: contributor
  4. laanwj commented at 11:29 AM on January 10, 2014: member

    Duplicate of #1044

  5. laanwj closed this on Jan 10, 2014

  6. DrahtBot locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-21 18:16 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me