Page MenuHomePhabricator
Feed Advanced Search

Jan 21 2021

majcosta accepted D9011: [test] Integration tests for unbroadcast functionality.
Jan 21 2021, 13:35
majcosta updated the summary of D9011: [test] Integration tests for unbroadcast functionality.
Jan 21 2021, 13:20
majcosta accepted D9010: [refactor/test] Extract P2PTxInvStore into test framework.
Jan 21 2021, 13:18
majcosta accepted D9008: [p2p] Reattempt initial send of unbroadcast transactions.
Jan 21 2021, 13:14
majcosta accepted D9007: [util] Add method that returns random time in milliseconds.
Jan 21 2021, 13:11
majcosta requested changes to D9006: [mempool] Track "unbroadcast" transactions.
Jan 21 2021, 13:09
majcosta accepted D9005: tests: Add fuzzing harness for Golomb-Rice coding (GolombRiceEncode/GolombRiceDecode).
Jan 21 2021, 13:01
majcosta accepted D9004: fuzz: fix unused variable compiler warning.
Jan 21 2021, 12:43
majcosta accepted D9003: Add fuzzer version of randomized prevector test.
Jan 21 2021, 12:41
majcosta closed D8992: [backport#18754] bench: add CAddrMan benchmarks.
Jan 21 2021, 12:30
majcosta committed rABC8b0b3a190975: [backport#18754] bench: add CAddrMan benchmarks (authored by MarcoFalke <falke.marco@gmail.com>).
[backport#18754] bench: add CAddrMan benchmarks
Jan 21 2021, 12:30
majcosta closed D8993: [backport#18669] log: Use Join() helper when listing log categories.
Jan 21 2021, 12:29
majcosta committed rABCf9ec355922bb: [backport#18669] log: Use Join() helper when listing log categories (authored by MarcoFalke <falke.marco@gmail.com>).
[backport#18669] log: Use Join() helper when listing log categories
Jan 21 2021, 12:28
majcosta closed D8994: [backport#18777] wallet: Recommend absolute path for dumpwallet.
Jan 21 2021, 12:26
majcosta committed rABCd0fa5ad23b7c: [backport#18777] wallet: Recommend absolute path for dumpwallet (authored by MarcoFalke <falke.marco@gmail.com>).
[backport#18777] wallet: Recommend absolute path for dumpwallet
Jan 21 2021, 12:26
majcosta updated the diff for D8992: [backport#18754] bench: add CAddrMan benchmarks.

addressed feedback

Jan 21 2021, 12:03
majcosta accepted D8997: [CI] Elect more builds to run on diff depending on simple heuristic.
Jan 21 2021, 11:58
majcosta abandoned D8990: [backport#18665] Do not expose and consider -logthreadnames when it does not work.

oh, looks like this one isn't helpful then. abandoning.

Jan 21 2021, 11:57
teamcity awarded D8992: [backport#18754] bench: add CAddrMan benchmarks a Haypence token.
Jan 21 2021, 08:55
majcosta updated the diff for D8992: [backport#18754] bench: add CAddrMan benchmarks.

rebase

Jan 21 2021, 00:35

Jan 20 2021

majcosta requested review of D8994: [backport#18777] wallet: Recommend absolute path for dumpwallet.
Jan 20 2021, 19:29
majcosta requested review of D8993: [backport#18669] log: Use Join() helper when listing log categories.
Jan 20 2021, 19:23
majcosta requested review of D8992: [backport#18754] bench: add CAddrMan benchmarks.
Jan 20 2021, 18:52
majcosta accepted D8988: test: use zero-argument super() shortcut (Python 3.0+).
Jan 20 2021, 18:19
majcosta accepted D8991: test: Fix intermittent failure in wallet_importmulti.
Jan 20 2021, 18:01
majcosta accepted D8989: test: Remove raw-tx byte juggling in mempool_reorg.
Jan 20 2021, 17:58
majcosta accepted D8987: tests: Add fuzzing harnesses for classes/functions in policy/fees.h, checkqueue.h and cuckoocache.h. Add fuzzing coverage..
Jan 20 2021, 17:35
majcosta requested review of D8990: [backport#18665] Do not expose and consider -logthreadnames when it does not work.
Jan 20 2021, 16:57
majcosta accepted D8979: tests: Add fuzzing harnesses for functions/classes in chain.h and protocol.h.
Jan 20 2021, 16:47
majcosta requested changes to D8979: tests: Add fuzzing harnesses for functions/classes in chain.h and protocol.h.
Jan 20 2021, 15:18

Jan 18 2021

majcosta closed D8938: [backport#16945#18181] refactor: introduce CChainState::GetCoinsCacheSizeState [plus fix].
Jan 18 2021, 13:06
majcosta committed rABC7680ec0096b1: [backport#16945#18181] refactor: introduce CChainState::GetCoinsCacheSizeState… (authored by Wladimir J. van der Laan <laanwj@protonmail.com>).
[backport#16945#18181] refactor: introduce CChainState::GetCoinsCacheSizeState…
Jan 18 2021, 13:05
teamcity awarded D8938: [backport#16945#18181] refactor: introduce CChainState::GetCoinsCacheSizeState [plus fix] a Doubloon token.
Jan 18 2021, 11:32
majcosta updated the diff for D8938: [backport#16945#18181] refactor: introduce CChainState::GetCoinsCacheSizeState [plus fix].

removed commented leftover debugging line and adapted the test for Coin::DynamicMemoryUsage() on 32-bit

Jan 18 2021, 11:12
teamcity awarded D8938: [backport#16945#18181] refactor: introduce CChainState::GetCoinsCacheSizeState [plus fix] a Piece of Eight token.
Jan 18 2021, 08:54
teamcity awarded D8938: [backport#16945#18181] refactor: introduce CChainState::GetCoinsCacheSizeState [plus fix] a Haypence token.
Jan 18 2021, 08:38

Jan 17 2021

majcosta updated the summary of D8938: [backport#16945#18181] refactor: introduce CChainState::GetCoinsCacheSizeState [plus fix].
Jan 17 2021, 12:11
majcosta accepted D8937: refactor: Remove unused methods CBloomFilter::reset()/clear().
Jan 17 2021, 10:37
majcosta accepted D8936: tests: have coins simulation test also use CCoinsViewDB.
Jan 17 2021, 10:35
majcosta accepted D8935: script: Disallow silent bool -> CScript conversion.
Jan 17 2021, 10:33
majcosta requested review of D8938: [backport#16945#18181] refactor: introduce CChainState::GetCoinsCacheSizeState [plus fix].
Jan 17 2021, 10:26

Jan 15 2021

majcosta closed D8921: [backport#16380] Remove unused bits from the service flags enum.
Jan 15 2021, 16:23
majcosta committed rABCb051c69478dd: [backport#16380] Remove unused bits from the service flags enum (authored by fanquake <fanquake@gmail.com>).
[backport#16380] Remove unused bits from the service flags enum
Jan 15 2021, 16:23
majcosta accepted D8932: test: Verify findCommonAncestor always initializes outputs.
Jan 15 2021, 15:35
majcosta accepted D8929: test: Add various low-level p2p tests.
Jan 15 2021, 14:45
majcosta accepted D8931: test: coverage improvements in interface-bitcoin_cli.py.
Jan 15 2021, 14:43
majcosta accepted D8930: [doc] Update thread information in developer docs.
Jan 15 2021, 14:20
majcosta requested changes to D8929: test: Add various low-level p2p tests.
Jan 15 2021, 13:53
majcosta updated the diff for D8921: [backport#16380] Remove unused bits from the service flags enum.

corrected date of removal and removed declaration in messages.py

Jan 15 2021, 11:31

Jan 14 2021

majcosta accepted D8918: Deprecate the `setexcessiveblock` RPC.
Jan 14 2021, 17:43
majcosta requested review of D8921: [backport#16380] Remove unused bits from the service flags enum.
Jan 14 2021, 17:42
majcosta accepted D8920: test: Remove REJECT message code.
Jan 14 2021, 17:15
majcosta accepted D8919: rpc: Remove deprecated migration code.
Jan 14 2021, 17:13
majcosta closed D8914: [backport#17453] gui: Fix intro dialog labels when the prune button is toggled.
Jan 14 2021, 15:38
majcosta committed rABCfbd564ae6a78: [backport#17453] gui: Fix intro dialog labels when the prune button is toggled (authored by Jonas Schnelli <dev@jonasschnelli.ch>).
[backport#17453] gui: Fix intro dialog labels when the prune button is toggled
Jan 14 2021, 15:38
majcosta closed D8916: [backport#15874] Resolve the qt/guiutil <-> qt/optionsmodel CD.
Jan 14 2021, 15:38
majcosta committed rABCfc8c7fd5b1fa: [backport#15874] Resolve the qt/guiutil <-> qt/optionsmodel CD (authored by Jonas Schnelli <dev@jonasschnelli.ch>).
[backport#15874] Resolve the qt/guiutil <-> qt/optionsmodel CD
Jan 14 2021, 15:38
majcosta updated the diff for D8914: [backport#17453] gui: Fix intro dialog labels when the prune button is toggled.

rebase on top of D8916 which fixes the circular dependency h/t @PiRK

Jan 14 2021, 14:41
majcosta closed D8913: [backport#17696] qt: Force set nPruneSize in QSettings after the intro dialog.
Jan 14 2021, 14:39
majcosta committed rABC6a7934983f7f: [backport#17696] qt: Force set nPruneSize in QSettings after the intro dialog (authored by fanquake <fanquake@gmail.com>).
[backport#17696] qt: Force set nPruneSize in QSettings after the intro dialog
Jan 14 2021, 14:39
majcosta accepted D8915: Add a test for the banscore deprecation.
Jan 14 2021, 14:37
majcosta requested review of D8916: [backport#15874] Resolve the qt/guiutil <-> qt/optionsmodel CD.
Jan 14 2021, 14:37
majcosta accepted D8912: cli -getinfo: use getbalances instead of deprecated getwalletinfo balance.
Jan 14 2021, 14:37
majcosta added a comment to D8912: cli -getinfo: use getbalances instead of deprecated getwalletinfo balance.

also "Backport of PR18574" is missing from summary

Jan 14 2021, 13:54
majcosta requested changes to D8912: cli -getinfo: use getbalances instead of deprecated getwalletinfo balance.
Jan 14 2021, 13:53
majcosta accepted D8910: Remove PID file at the very end.
Jan 14 2021, 13:50
majcosta accepted D8911: rpc: Make verifychain default values static, not depend on global args.
Jan 14 2021, 13:49
majcosta updated the summary of D8914: [backport#17453] gui: Fix intro dialog labels when the prune button is toggled.
Jan 14 2021, 13:03
majcosta requested review of D8914: [backport#17453] gui: Fix intro dialog labels when the prune button is toggled.
Jan 14 2021, 13:03
majcosta requested review of D8913: [backport#17696] qt: Force set nPruneSize in QSettings after the intro dialog.
Jan 14 2021, 13:02
majcosta requested changes to D8910: Remove PID file at the very end.

tests failing

Jan 14 2021, 11:58
majcosta accepted D8908: Fix a -Wunused-variable in the locale fuzzer.
Jan 14 2021, 11:34
majcosta accepted D8905: Moving dev dependencies to dev, removing unused deps, updates some of them.
Jan 14 2021, 11:29
majcosta retitled D8905: Moving dev dependencies to dev, removing unused deps, updates some of them from Moving dev dependencies to dev, removing unused deps to Moving dev dependencies to dev, removing unused deps, updates some of them.
Jan 14 2021, 11:02
majcosta requested changes to D8905: Moving dev dependencies to dev, removing unused deps, updates some of them.
Jan 14 2021, 04:01

Jan 13 2021

majcosta accepted D8904: [Cashtab] bump extension version to support new backend per D8885.
Jan 13 2021, 16:06
majcosta accepted D8901: [CashTab] Update react-copy-to-clipboard.
Jan 13 2021, 15:20
majcosta accepted D8902: rpc: Make rpc documentation not depend on rpc args.
Jan 13 2021, 15:08
majcosta closed D8762: [backport#16658] validation: Rename CheckInputs to CheckInputScripts.
Jan 13 2021, 14:44
majcosta committed rABC925120cac2cb: [backport#16658] validation: Rename CheckInputs to CheckInputScripts (authored by MarcoFalke <falke.marco@gmail.com>).
[backport#16658] validation: Rename CheckInputs to CheckInputScripts
Jan 13 2021, 14:44
majcosta requested review of D8762: [backport#16658] validation: Rename CheckInputs to CheckInputScripts.

it looks like D3878 and D3875 diverged from Core so that the tests don't rely on whether "CheckInputs" or "CheckInputScripts" gets printed to the debug log

Jan 13 2021, 13:58

Jan 12 2021

majcosta accepted D8884: rpc: fix broken RPCExamples for waitforblock(height).
Jan 12 2021, 18:31
majcosta accepted D8878: Replace std::to_string with locale-independent alternative.
Jan 12 2021, 15:18
majcosta accepted D8881: test: Fix restart node race.
Jan 12 2021, 13:55
majcosta accepted D8872: test: add unit test for non-standard "scriptsig-not-pushonly" txs.
Jan 12 2021, 11:16
majcosta requested changes to D8878: Replace std::to_string with locale-independent alternative.
Jan 12 2021, 11:04
majcosta requested changes to D8872: test: add unit test for non-standard "scriptsig-not-pushonly" txs.
Jan 12 2021, 10:32

Jan 11 2021

majcosta accepted D8876: gui: display Mapped AS in peers info window.
Jan 11 2021, 17:36
majcosta requested changes to D8872: test: add unit test for non-standard "scriptsig-not-pushonly" txs.

this is PR17720 right? missing in the summary

Jan 11 2021, 17:30
majcosta accepted D8875: [CI] Add cashtab coverage report to the coverage panel.
Jan 11 2021, 17:19
majcosta accepted D8874: [CI] Generate a CashTab coverage summary report.
Jan 11 2021, 16:48
majcosta accepted D8873: [Cashtab] remove duplicate key from test mock.
Jan 11 2021, 15:19
majcosta accepted D8870: [Cashtab] extension with new pkgs, version bump.
Jan 11 2021, 15:10
majcosta accepted D8871: gui: Avoid redundant tx status updates.
Jan 11 2021, 14:43
majcosta requested changes to D8870: [Cashtab] extension with new pkgs, version bump.

tests are failing

Jan 11 2021, 13:21
majcosta accepted D8867: [validation] Remove NotifyEntryRemoved callback from ConnectTrace.
Jan 11 2021, 13:20
majcosta accepted D8869: [mempool] Remove NotifyEntryAdded and NotifyEntryRemoved callbacks.
Jan 11 2021, 13:20
majcosta requested changes to D8867: [validation] Remove NotifyEntryRemoved callback from ConnectTrace.
Jan 11 2021, 12:31
majcosta added a comment to D8867: [validation] Remove NotifyEntryRemoved callback from ConnectTrace.
In D8867#203607, @PiRK wrote:

I think the clang warning that is treated as an error goes away with the following commit (D8868). Can I just make sure I land them in rapid succession, or do I need to squash them?

Jan 11 2021, 12:25
majcosta added a comment to D8867: [validation] Remove NotifyEntryRemoved callback from ConnectTrace.
Jan 11 2021, 12:23