Skip to content

Conversation

@Rutkouski
Copy link

@Rutkouski Rutkouski commented Feb 3, 2018

No description provided.

CryptoManiac and others added 30 commits February 11, 2016 01:43
GCC doesn't like 0LL here.
Fix V668 PVS Studio - deal with pointer and NULL
Fix V595 PVS Studio - the pointer was utilized before it was verified…
Save generated key pairs in the wallet.dat.

Due to lack of time for a proper debugging the generation of malleable keys has been disabled on the main network. Of course it will be enabled in the near future.
Оставлю как якорь, подобных мест в коде много, по возможности можно будет переписать.
fDetachDB missed in constructor
@sipa sipa closed this Feb 3, 2018
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Nov 9, 2018
This is meant to be a direct translation of the bash script,
with the difference that it retrieves the PR title from github,
thus creating pull messages like:

    Merge bitcoin#12345: Expose transaction temperature over RPC
jasonbcox added a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Sep 21, 2020
…s in diff summaries

Summary:
We sometimes see variants like the following:
* Backport of [[bitcoin/bitcoin#12345 | PR12345]]
* Backport of PR 12345
* Backport of PR#12345
* Backport of PR #12345

Only the first one is automatically linked. We can detect these and both fix and link them
rather than ignoring them. The rationale for fixing them instead of leaving as-is is because
PRxxxxx is easier to grep if they are consistent in all patch summaries.

Other variants of the above are welcome. I simply picked these because we see them the most.

Test Plan: `pytest`

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Subscribers: deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D7489
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants