minor txindex documentation improvement #5047

pull cozz wants to merge 1 commits into bitcoin:master from cozz:cozz7 changing 2 files +5 −3

cozz commented at 11:34 PM on October 5, 2014: contributor

No description provided.
sipa commented at 12:07 AM on October 6, 2014: member

I would even formulate it as "makes getrawtransaction work" - the speed advantage is small compared to it otherwise being completely unreliable.
cozz force-pushed on Oct 6, 2014
cozz commented at 1:02 AM on October 6, 2014: contributor

ok, updated. I just think, there is documentation missing, for what really the txindex is there for.
minor txindex documentation improvement 3e61eb9c51

in src/init.cpp:None in e5cb9ee4a7 outdated

 236 | @@ -237,7 +237,7 @@ std::string HelpMessage(HelpMessageMode mode)
 237 |  #if !defined(WIN32)
 238 |      strUsage += "  -sysperms              " + _("Create new files with system default permissions, instead of umask 077 (only effective with disabled wallet functionality)") + "\n";
 239 |  #endif
 240 | -    strUsage += "  -txindex               " + _("Maintain a full transaction index (default: 0)") + "\n";
 241 | +    strUsage += "  -txindex               " + _("Maintain a full transaction index (default: 0)") + _(" (Used by the getrawtransaction rpc call)") + "\n";

laanwj commented at 2:33 AM on October 6, 2014:

just change the description to "Maintain a full transaction index as used by the getrawtransaction rpc call". No need to to a () after the main () :)

cozz force-pushed on Oct 6, 2014
cozz commented at 4:54 PM on October 6, 2014: contributor

@laanwj fixed
laanwj merged this on Oct 7, 2014
laanwj closed this on Oct 7, 2014
laanwj referenced this in commit 3eaa9462f8 on Oct 7, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors

cozz

sipa

laanwj