< jayd3e> why does bitcoind.cpp use fprintf(stdout, "%s", strUsage.c_str()); instead of printf()? Isn't fprintf(stdout,..) equivalent to printf?
< sipa> in a long gone ancient past, bitcoind had its own printf function
< sipa> which has since been renamed and reworked into LogPrint and LogPrintf
< sipa> so we could turn that fprintf into a printf, indeed
< sipa> send a pull request :)
< jayd3e> cool
< jayd3e> will do, for brevity sake
< GitHub5> [bitcoin] 21E14 opened pull request #7263: Double semicolon cleanup (master...doubles) https://github.com/bitcoin/bitcoin/pull/7263
< jayd3e> sipa: what's the data dir used for?
< jayd3e> sipa: nvm got it
< GitHub148> [bitcoin] CodeShark opened pull request #7264: Segwit nits (master...segwit_nits) https://github.com/bitcoin/bitcoin/pull/7264
< CodeShark> argh!
< GitHub18> [bitcoin] CodeShark closed pull request #7264: Segwit nits (master...segwit_nits) https://github.com/bitcoin/bitcoin/pull/7264
< CodeShark> sorry, my mouse sucks
< jayd3e> what's the significance of the ampersand before a function name? such as in this line: CChainParams &Params(CBaseChainParams::Network network)
< jayd3e> from chainparams.cpp
< sipa> jayd3e: no offence, but this is not the place to learn C++
< jayd3e> yeah true. I've just been going through the code, and wanted to interact with the community at the same time
< instagibbs> Googling has been effective for me reading Core code.
< jayd3e> it's also nice to be able to reference the codebase
< Luke-Jr> jayd3e: #bitcoin
< MarcoFalke> or stackoverflow ;)
< GitHub31> [bitcoin] zauguin opened pull request #7266: Removed comment about IsStandard for P2SH scripts (master...zauguin-patch-1) https://github.com/bitcoin/bitcoin/pull/7266
< phantomcircuit> sipa, SetReachable, why the special casing for ipv6/ipv4 ?
< phantomcircuit> you can potentially be ipv6 w/o ipv4 ... right?
< sipa> no clue!
< phantomcircuit> sipa, lol you wrote it
< phantomcircuit> to be fair it seems you wrote it in 2012
< GitHub83> [bitcoin] doged opened pull request #7267: Update Makefile.am (master...master) https://github.com/bitcoin/bitcoin/pull/7267
< GitHub89> [bitcoin] doged closed pull request #7267: Update Makefile.am (master...master) https://github.com/bitcoin/bitcoin/pull/7267