Page MenuHomePhabricator
Feed Advanced Search

Sep 18 2019

nakihito planned changes to D3882: Merge #12920: test: Fix sign for expected values.

Going to separate the the code into two parts: the code that can be landed without dependencies and the code that is dependent on D3689.

Sep 18 2019, 15:57

Sep 17 2019

nakihito requested review of D4102: Merge #13431: validation: count blocks correctly for check level < 3.
Sep 17 2019, 23:32
nakihito requested review of D4103: Merge #13428: validation: check the specified number of blocks (off-by-one).
Sep 17 2019, 23:32
nakihito updated the summary of D4102: Merge #13431: validation: count blocks correctly for check level < 3.
Sep 17 2019, 23:07
nakihito planned changes to D4102: Merge #13431: validation: count blocks correctly for check level < 3.
Sep 17 2019, 23:06
nakihito updated the diff for D4102: Merge #13431: validation: count blocks correctly for check level < 3.

Rebase off D4103 to fix off by one error.

Sep 17 2019, 23:06
nakihito planned changes to D4103: Merge #13428: validation: check the specified number of blocks (off-by-one).
Sep 17 2019, 23:05
nakihito created D4103: Merge #13428: validation: check the specified number of blocks (off-by-one).
Sep 17 2019, 23:05
nakihito planned changes to D4102: Merge #13431: validation: count blocks correctly for check level < 3.
Sep 17 2019, 23:03
nakihito created D4102: Merge #13431: validation: count blocks correctly for check level < 3.
Sep 17 2019, 23:03
nakihito requested review of D3464: [rpc] Add logging RPC.
Sep 17 2019, 18:45
nakihito updated the diff for D3968: Merge #12885: Reduce implementation code inside CScript.

Changed location of FindandDelete() and restored newline.

Sep 17 2019, 18:40
nakihito planned changes to D3464: [rpc] Add logging RPC.
Sep 17 2019, 00:40
nakihito updated the diff for D3464: [rpc] Add logging RPC.

Fix comment, removed unnecessary #include, and fixed libevent filter.

Sep 17 2019, 00:40

Sep 13 2019

nakihito accepted D4061: [CI] Fix missing parameters transfer from ibd.sh to bitcoind.
Sep 13 2019, 00:38

Sep 12 2019

nakihito updated the diff for D3805: Merge #12716: Fix typos and cleanup in various files.

Fixed nit.

Sep 12 2019, 21:53

Sep 11 2019

nakihito updated the diff for D4045: Revert change to ActivateBestChain() comments made in D1182.

Removed punctuation at end of comment.

Sep 11 2019, 23:50
nakihito planned changes to D4026: Merge #13481: doc: Rewrite some validation docs as lock annotations.
Sep 11 2019, 21:43
nakihito planned changes to D4029: Merge #13012: [doc] Add comments for chainparams.h, validation.cpp.
Sep 11 2019, 20:56
nakihito updated the diff for D4029: Merge #13012: [doc] Add comments for chainparams.h, validation.cpp.

Rebased off D4045.

Sep 11 2019, 20:56
nakihito planned changes to D3935: Merge #11411: script: Change SignatureHash input index check to an assert..
Sep 11 2019, 20:43
nakihito updated the summary of D4029: Merge #13012: [doc] Add comments for chainparams.h, validation.cpp.
Sep 11 2019, 20:42
nakihito created D4045: Revert change to ActivateBestChain() comments made in D1182.
Sep 11 2019, 20:40
nakihito updated the diff for D3750: Merge #12159: Use the character based overload for std::string::find..

Fixed nit.

Sep 11 2019, 20:40
nakihito planned changes to D3689: Merge #11293: Deduplicate CMerkleBlock construction code, add test coverage.
Sep 11 2019, 20:12
nakihito updated the diff for D3750: Merge #12159: Use the character based overload for std::string::find..

Fixed remaining instance in wallet.h.

Sep 11 2019, 18:27
nakihito updated the diff for D4042: Merge #11220: Check specific validation error in miner tests.

Fixed comment.

Sep 11 2019, 18:10
nakihito requested review of D4042: Merge #11220: Check specific validation error in miner tests.
Sep 11 2019, 17:50
nakihito updated the summary of D4042: Merge #11220: Check specific validation error in miner tests.
Sep 11 2019, 17:37
nakihito planned changes to D4042: Merge #11220: Check specific validation error in miner tests.
Sep 11 2019, 17:36
nakihito created D4042: Merge #11220: Check specific validation error in miner tests.
Sep 11 2019, 17:36

Sep 10 2019

nakihito requested review of D4029: Merge #13012: [doc] Add comments for chainparams.h, validation.cpp.
Sep 10 2019, 21:26
nakihito accepted D3914: Merge #12853: qa: Match full plain text by default.
Sep 10 2019, 20:26
nakihito added inline comments to D4029: Merge #13012: [doc] Add comments for chainparams.h, validation.cpp.
Sep 10 2019, 20:18
nakihito updated the summary of D4026: Merge #13481: doc: Rewrite some validation docs as lock annotations.
Sep 10 2019, 19:21
nakihito updated the diff for D4026: Merge #13481: doc: Rewrite some validation docs as lock annotations.

Rebased off D4029 and added comment changes.

Sep 10 2019, 19:20
nakihito created D4029: Merge #13012: [doc] Add comments for chainparams.h, validation.cpp.
Sep 10 2019, 19:17
nakihito planned changes to D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.
Sep 10 2019, 01:21
nakihito added a comment to D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.

Will rebase off D4026 soon.

Sep 10 2019, 01:14
nakihito updated the summary of D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.
Sep 10 2019, 00:27
nakihito created D4026: Merge #13481: doc: Rewrite some validation docs as lock annotations.
Sep 10 2019, 00:26

Sep 9 2019

nakihito requested review of D3750: Merge #12159: Use the character based overload for std::string::find..
Sep 9 2019, 18:25
nakihito updated the test plan for D3937: Merge #12928: qt: Initialize non-static class members that were previously neither initialized where defined nor in constructor.
Sep 9 2019, 17:28
nakihito requested review of D4021: Merge #13149: Handle unsuccessful fseek(...):s.
Sep 9 2019, 17:06
nakihito updated the summary of D4021: Merge #13149: Handle unsuccessful fseek(...):s.
Sep 9 2019, 17:06

Sep 6 2019

nakihito added a comment to D4022: Merge #12503: [RPC] createmultisig no longer takes addresses.

using addresses in createmultisig was deprecated in D2002 and the deprecated code was removed in D3978.

Sep 6 2019, 18:35
nakihito created D4022: Merge #12503: [RPC] createmultisig no longer takes addresses.
Sep 6 2019, 18:33
nakihito created D4021: Merge #13149: Handle unsuccessful fseek(...):s.
Sep 6 2019, 18:22
nakihito updated the diff for D3934: Merge #11338: qt: Backup former GUI settings on `-resetguisettings`.

Fixed comment formatting and removed extra new line.

Sep 6 2019, 17:57
nakihito created D4020: Merge #13658: [moveonly] Extract RescanWallet to handle a simple rescan.
Sep 6 2019, 17:55
nakihito requested review of D3542: Merge #10537: Few Minor per-utxo assert-semantics re-adds and tweak.
Sep 6 2019, 17:16
nakihito requested review of D3937: Merge #12928: qt: Initialize non-static class members that were previously neither initialized where defined nor in constructor.
Sep 6 2019, 17:14

Sep 5 2019

nakihito updated the summary of D3950: Merge #12953: Deprecate accounts.
Sep 5 2019, 22:48
nakihito added inline comments to D3997: Merge #11050: Avoid treating null RPC arguments different from missing arguments.
Sep 5 2019, 19:20
nakihito updated the summary of D3997: Merge #11050: Avoid treating null RPC arguments different from missing arguments.
Sep 5 2019, 19:16
nakihito updated the diff for D3997: Merge #11050: Avoid treating null RPC arguments different from missing arguments.

Corrected nits, rebased off D3464, and added changes to logging().

Sep 5 2019, 19:15

Sep 4 2019

nakihito updated the summary of D3981: Merge #13081: wallet: Add compile time checking for cs_wallet runtime locking assertions.
Sep 4 2019, 22:09
nakihito updated the diff for D3981: Merge #13081: wallet: Add compile time checking for cs_wallet runtime locking assertions.

Rebased off D3998.

Sep 4 2019, 22:08
nakihito created D3998: Merge #11044: [wallet] Keypool topup cleanups.
Sep 4 2019, 21:50
nakihito planned changes to D3950: Merge #12953: Deprecate accounts.

Nits fixed. Will rebase after D3997 is landed.

Sep 4 2019, 21:44
nakihito added a comment to D3997: Merge #11050: Avoid treating null RPC arguments different from missing arguments.

rpc/blockchain.cpp changes were made here: D954
rpc/mining.cpp changes were excluded because we did not backport much of Core's fee code.
Changes to rpc/misc.cpp that were excluded were to outdated logging code.
signrawtransaction() was removed after being deprecated: D3978

Sep 4 2019, 21:36
nakihito created D3997: Merge #11050: Avoid treating null RPC arguments different from missing arguments.
Sep 4 2019, 21:17

Sep 3 2019

nakihito updated the diff for D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.

Rebased and added missing EXCLUSIVE_LOCKS_REQUIRED() to validation.cpp

Sep 3 2019, 19:51
nakihito updated the diff for D3981: Merge #13081: wallet: Add compile time checking for cs_wallet runtime locking assertions.

Added an instance of EXCLUSIVE_LOCKS_REQUIRED() to wallet/wallet.cpp.

Sep 3 2019, 19:47
nakihito updated the diff for D3981: Merge #13081: wallet: Add compile time checking for cs_wallet runtime locking assertions.

Rebased.

Sep 3 2019, 19:16
nakihito updated the test plan for D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.
Sep 3 2019, 19:10
nakihito updated the test plan for D3981: Merge #13081: wallet: Add compile time checking for cs_wallet runtime locking assertions.
Sep 3 2019, 19:06
nakihito requested review of D3980: Merge #13077: Add compile time checking for all cs_KeyStore runtime locking assertions.

Fixed test method.

Sep 3 2019, 19:06
nakihito updated the test plan for D3980: Merge #13077: Add compile time checking for all cs_KeyStore runtime locking assertions.
Sep 3 2019, 19:06

Aug 31 2019

nakihito updated the summary of D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.
Aug 31 2019, 00:39
nakihito updated the summary of D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.
Aug 31 2019, 00:38
nakihito created D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.
Aug 31 2019, 00:38
nakihito created D3981: Merge #13081: wallet: Add compile time checking for cs_wallet runtime locking assertions.
Aug 31 2019, 00:37
nakihito created D3980: Merge #13077: Add compile time checking for all cs_KeyStore runtime locking assertions.
Aug 31 2019, 00:34

Aug 30 2019

nakihito requested review of D3976: Merge #13148: logging: Fix potential use-after-free in LogPrintStr(...).

Fix completed by D3979.

Aug 30 2019, 20:54
nakihito updated the summary of D3979: Merge #13159: Don't close old debug log file handle prematurely when trying to re-open (on SIGHUP).
Aug 30 2019, 20:53
nakihito created D3979: Merge #13159: Don't close old debug log file handle prematurely when trying to re-open (on SIGHUP).
Aug 30 2019, 20:53
nakihito updated the diff for D3968: Merge #12885: Reduce implementation code inside CScript.

Fixed variable type comparison warning.

Aug 30 2019, 18:47
nakihito updated the diff for D3950: Merge #12953: Deprecate accounts.

Rebased.

Aug 30 2019, 18:10
nakihito updated the diff for D3949: Merge #12892: [wallet] [rpc] introduce 'label' API for wallet.

Rebased and moved label and account function blocks inline with Core's placement.

Aug 30 2019, 17:51
nakihito planned changes to D3976: Merge #13148: logging: Fix potential use-after-free in LogPrintStr(...).

Not a complete fix. https://github.com/bitcoin/bitcoin/pull/13159 fixes remaining issues.

Aug 30 2019, 01:44
nakihito created D3976: Merge #13148: logging: Fix potential use-after-free in LogPrintStr(...).
Aug 30 2019, 01:32

Aug 29 2019

nakihito created D3975: Merge #12507: Interrupt rescan on shutdown request.
Aug 29 2019, 23:04
nakihito updated the diff for D3949: Merge #12892: [wallet] [rpc] introduce 'label' API for wallet.

Addressed nits and reverted timing.json change.

Aug 29 2019, 20:52
nakihito updated the diff for D3968: Merge #12885: Reduce implementation code inside CScript.

Fixed variable typing and added braces.

Aug 29 2019, 20:40

Aug 28 2019

nakihito added inline comments to D3968: Merge #12885: Reduce implementation code inside CScript.
Aug 28 2019, 23:19
nakihito updated the diff for D3968: Merge #12885: Reduce implementation code inside CScript.

Removed typo.

Aug 28 2019, 23:19
nakihito created D3968: Merge #12885: Reduce implementation code inside CScript.
Aug 28 2019, 23:16
nakihito updated the diff for D3954: Merge #12923: util: Pass pthread_self() to pthread_setschedparam instead of 0.

Rebase.

Aug 28 2019, 19:16
nakihito updated the diff for D3953: Merge #12618: Set SCHED_BATCH priority on the loadblk thread..

util.cpp change was accidentally skipped.

Aug 28 2019, 19:13
nakihito updated the diff for D3953: Merge #12618: Set SCHED_BATCH priority on the loadblk thread..

function(void) -> function().

Aug 28 2019, 19:12
nakihito updated the diff for D3954: Merge #12923: util: Pass pthread_self() to pthread_setschedparam instead of 0.

Rebased.

Aug 28 2019, 19:10
nakihito updated the diff for D3953: Merge #12618: Set SCHED_BATCH priority on the loadblk thread..

Fixed function(void) -> function().

Aug 28 2019, 18:35
nakihito abandoned D3696: Merge #10036: Fix init README format to render correctly on github.

I guess this was landed as D3843 while this was waiting for review.

Aug 28 2019, 18:02
nakihito requested review of D3897: Merge #12709: [wallet] shuffle sendmany recipients ordering.
Aug 28 2019, 00:57
nakihito added a comment to D3953: Merge #12618: Set SCHED_BATCH priority on the loadblk thread..

https://github.com/bitcoin/bitcoin/issues/12915
This is supposed to introduce the above bug, but I could not build with the same configure options to test. The fix is made in D3954.

Aug 28 2019, 00:54
nakihito updated the summary of D3954: Merge #12923: util: Pass pthread_self() to pthread_setschedparam instead of 0.
Aug 28 2019, 00:52
nakihito created D3954: Merge #12923: util: Pass pthread_self() to pthread_setschedparam instead of 0.
Aug 28 2019, 00:52
nakihito created D3953: Merge #12618: Set SCHED_BATCH priority on the loadblk thread..
Aug 28 2019, 00:50

Aug 27 2019

nakihito created D3952: Merge #12702: [wallet] [rpc] [doc] importprivkey: hint about importmulti.
Aug 27 2019, 20:31