Paths

Table of Contentst

Always use the parallel check codepath for block validation
ClosedPublic
Actions

Authored by deadalnix on Sep 4 2017, 20:23.

Details

Reviewers

freetrader
CCulianu
sickpig
dagurval

Group Reviewers

Restricted Project

Commits

rSTAGING8ed800afad05: Always use the parallel check codepath for block validation
rABC8ed800afad05: Always use the parallel check codepath for block validation

Summary

Less codepath, more fun. We also had a bug linked to only one of the two path being tested.

Test Plan

Synced the testnet with -par=0 and -par=1

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

deadalnix created this revision.Sep 4 2017, 20:23

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 4 2017, 20:23

sickpig added a subscriber: sickpig.Sep 5 2017, 09:59

sickpig added inline comments.

src/validation.cpp
1341 ↗	(On Diff #1311)	nit: it is a pretty simple change but I wonder it wouldn't be better to this kind of modifications in a separate diff and for the entire file (there are 22 instances of `unsigned int` in this file alone)

Accepted modulo the unsigned int unrelated change (but I'm good in either case) .

After this chance there is nothing but init.cpp using nScriptCheckThreads. How about making it a local variable there? One less global variable.

This revision now requires changes to proceed.Sep 5 2017, 22:27

Rebase.

@dagurval, removing the global variable and querying GetArg instead doesn't make the value any less global.

Harbormaster completed remote builds in B836: Diff 1325.Sep 5 2017, 23:17

In D521#9312, @deadalnix wrote:

@dagurval, removing the global variable and querying GetArg instead doesn't make the value any less global.

You'll literally have one less global variable decleration in validation.h.

dagurval accepted this revision.Sep 6 2017, 21:51

This revision is now accepted and ready to land.Sep 6 2017, 21:51

Closed by commit rABC8ed800afad05: Always use the parallel check codepath for block validation (authored by deadalnix, committed by deadalnix). · Explain WhySep 6 2017, 21:53

This revision was automatically updated to reflect the committed changes.

jasonbcox mentioned this in D3875: Read more reject messages from debug logs in feature_dersig.py.Aug 14 2019, 18:23

jasonbcox mentioned this in D3878: Read more reject messages from debug logs in feature_cltv.py.Aug 14 2019, 19:09

jasonbcox mentioned this in rABCae9f9b6054c0: Read more reject messages from debug logs in feature_dersig.py.Aug 16 2019, 23:09

jasonbcox mentioned this in rABC958009f15f09: Read more reject messages from debug logs in feature_cltv.py.

jasonbcox mentioned this in rSTAGINGae9f9b6054c0: Read more reject messages from debug logs in feature_dersig.py.Sep 19 2019, 01:53

jasonbcox mentioned this in rSTAGING958009f15f09: Read more reject messages from debug logs in feature_cltv.py.

majcosta mentioned this in D7275: [backport#17342][refactor] Remove global int nScriptCheckThreads.Aug 31 2020, 20:09

majcosta mentioned this in rABC6b134a6895e1: [backport#17342][refactor] Remove global int nScriptCheckThreads.Sep 1 2020, 12:20

PiRK mentioned this in D7981: p2p: Remove BIP61 reject messages.Oct 19 2020, 15:22

Revision Contents
Changeset List

Path

Size

qa/

rpc-tests/

abc-rpc.py

3 lines

src/

validation.cpp

6 lines

Diff 1340

View Options

qa/rpc-tests/abc-rpc.py