Merge #8038: [qa, doc] Various minor fixes

fa83a5d [qa] wallet: Temporarily disable salvagewallet test (MarcoFalke)
fadd048 [doc] Link to clang-format in the developer notes (MarcoFalke)
fa72f7d [doc] Remove outdated line from listunspent RPC help, fix typo (MarcoFalke)
ac40ed7 Increase timeout waiting for pruned blk00000.dat (error10)
This commit is contained in:
MarcoFalke 2016-05-17 09:26:42 +02:00
commit e2bf830bb6
No known key found for this signature in database
GPG Key ID: 2D7F2372E50FE137
4 changed files with 9 additions and 8 deletions

View File

@ -5,7 +5,9 @@ Various coding styles have been used during the history of the codebase,
and the result is not very consistent. However, we're now trying to converge to and the result is not very consistent. However, we're now trying to converge to
a single style, so please use it in new code. Old code will be converted a single style, so please use it in new code. Old code will be converted
gradually. gradually.
- Basic rules specified in src/.clang-format. Use a recent clang-format-3.5 to format automatically. - Basic rules specified in [src/.clang-format](/src/.clang-format).
Use a recent clang-format to format automatically using one of the [dev scripts]
(/contrib/devtools/README.md#clang-formatpy).
- Braces on new lines for namespaces, classes, functions, methods. - Braces on new lines for namespaces, classes, functions, methods.
- Braces on the same line for everything else. - Braces on the same line for everything else.
- 4 space indentation (no tabs) for every block except namespaces. - 4 space indentation (no tabs) for every block except namespaces.

View File

@ -75,7 +75,7 @@ class PruneTest(BitcoinTestFramework):
waitstart = time.time() waitstart = time.time()
while os.path.isfile(self.prunedir+"blk00000.dat"): while os.path.isfile(self.prunedir+"blk00000.dat"):
time.sleep(0.1) time.sleep(0.1)
if time.time() - waitstart > 10: if time.time() - waitstart > 30:
raise AssertionError("blk00000.dat not pruned when it should be") raise AssertionError("blk00000.dat not pruned when it should be")
print("Success") print("Success")

View File

@ -306,7 +306,7 @@ class WalletTest (BitcoinTestFramework):
# Check that the txid and balance is found by node1 # Check that the txid and balance is found by node1
self.nodes[1].gettransaction(cbTxId) self.nodes[1].gettransaction(cbTxId)
#check if wallet or blochchain maintenance changes the balance # check if wallet or blockchain maintenance changes the balance
self.sync_all() self.sync_all()
blocks = self.nodes[0].generate(2) blocks = self.nodes[0].generate(2)
self.sync_all() self.sync_all()
@ -318,7 +318,8 @@ class WalletTest (BitcoinTestFramework):
'-reindex', '-reindex',
'-zapwallettxes=1', '-zapwallettxes=1',
'-zapwallettxes=2', '-zapwallettxes=2',
'-salvagewallet', # disabled until issue is fixed: https://github.com/bitcoin/bitcoin/issues/7463
# '-salvagewallet',
] ]
for m in maintenance: for m in maintenance:
print("check " + m) print("check " + m)
@ -338,4 +339,4 @@ class WalletTest (BitcoinTestFramework):
assert_equal(len(self.nodes[0].listsinceblock(blocks[1])["transactions"]), 0) assert_equal(len(self.nodes[0].listsinceblock(blocks[1])["transactions"]), 0)
if __name__ == '__main__': if __name__ == '__main__':
WalletTest ().main () WalletTest().main()

View File

@ -2233,7 +2233,7 @@ UniValue settxfee(const UniValue& params, bool fHelp)
"settxfee amount\n" "settxfee amount\n"
"\nSet the transaction fee per kB. Overwrites the paytxfee parameter.\n" "\nSet the transaction fee per kB. Overwrites the paytxfee parameter.\n"
"\nArguments:\n" "\nArguments:\n"
"1. amount (numeric or sting, required) The transaction fee in " + CURRENCY_UNIT + "/kB\n" "1. amount (numeric or string, required) The transaction fee in " + CURRENCY_UNIT + "/kB\n"
"\nResult\n" "\nResult\n"
"true|false (boolean) Returns true if successful\n" "true|false (boolean) Returns true if successful\n"
"\nExamples:\n" "\nExamples:\n"
@ -2328,8 +2328,6 @@ UniValue listunspent(const UniValue& params, bool fHelp)
"\nReturns array of unspent transaction outputs\n" "\nReturns array of unspent transaction outputs\n"
"with between minconf and maxconf (inclusive) confirmations.\n" "with between minconf and maxconf (inclusive) confirmations.\n"
"Optionally filter to only include txouts paid to specified addresses.\n" "Optionally filter to only include txouts paid to specified addresses.\n"
"Results are an array of Objects, each of which has:\n"
"{txid, vout, scriptPubKey, amount, confirmations}\n"
"\nArguments:\n" "\nArguments:\n"
"1. minconf (numeric, optional, default=1) The minimum confirmations to filter\n" "1. minconf (numeric, optional, default=1) The minimum confirmations to filter\n"
"2. maxconf (numeric, optional, default=9999999) The maximum confirmations to filter\n" "2. maxconf (numeric, optional, default=9999999) The maximum confirmations to filter\n"