Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport] 5.0 backports #2078

Merged
merged 86 commits into from
Dec 21, 2020
Merged

[Backport] 5.0 backports #2078

merged 86 commits into from
Dec 21, 2020

Conversation

Fuzzbawls
Copy link
Collaborator

@Fuzzbawls Fuzzbawls commented Dec 19, 2020

furszy and others added 30 commits December 16, 2020 15:53
Without the framework include, CMake isn't able to find the CFBundleGetMainBundle and relative classes/functions.

Github-Pull: PIVX-Project#2041
Rebased-From: 9afec2e
…ecrementally instead of incrementally.

Github-Pull: PIVX-Project#2046
Rebased-From: a1025e5
Avoids an upgrade warning from being triggered after 100 v8 blocks are
on chain. Moved the network upgrade consensus check further up.

Github-Pull: PIVX-Project#2052
Rebased-From: 710f480
also test that 64 bit integers are properly handled

Github-Pull: PIVX-Project#2059
Rebased-From: e0ba82b
…essarily scary.

* The "ERROR" was printed far too often during normal operation for what was not an error.

Coming from btc@00678bdb0aeb296456501f818faaa012a75ce18e

Github-Pull: PIVX-Project#2054
Rebased-From: 097322c
They were too noisy and not necessary for normal operation.

Coming from btc@0d9af79e5084dc2fb6a73abd9dd113dda4e993b4

Github-Pull: PIVX-Project#2054
Rebased-From: 2027d66
Before: ERROR: Error reading proxy response

Coming from btc@94fd1d8d53adceb80e5a41cc6438c7704aeac0f7

Github-Pull: PIVX-Project#2054
Rebased-From: fdec2e3
Coming from btc@bf9266e017b286c36e08fd09b91d9e39f14b2cf3

Github-Pull: PIVX-Project#2054
Rebased-From: a1b0017
Coming from btc@10ae7a7b2316f8052ec58ef237ce6dd987300900

Github-Pull: PIVX-Project#2054
Rebased-From: f62d152
Prior to this commit the value stored to `s` at initialization
was never read (in the case of STRERROR_R_CHAR_P).

Coming from btc@58446094519151e2e203944267e592eec2470643

Github-Pull: PIVX-Project#2054
Rebased-From: d46b369
…ping.

We are already caching it.

Github-Pull: PIVX-Project#2056
Rebased-From: c456202
…d budget vote.

We are already caching it.

[Masternode] CMasternodePayments::ProcessBlock, stop parsing the masternode private key string.

Stop parsing strMasterNodePrivKey when we already have it parsed in mnfinalbudget and mnbudgetvote

Github-Pull: PIVX-Project#2056
Rebased-From: 329e6ef
random-zebra and others added 7 commits December 18, 2020 21:35
(ZC_V2 upgrade activated with POS at block 201)

Github-Pull: PIVX-Project#2069
Rebased-From: 2ebb986
The combobox shouldn't get hide when there are transactions in the wallet, independently on the selected filter. If the user selects a type that it isn't in the wallet, the filter combobox will disappear, leaving the tx list empty and not allowing any other movement.

Github-Pull: PIVX-Project#2073
Rebased-From: 6dbe9ad
@Fuzzbawls Fuzzbawls added this to the 5.0.0 milestone Dec 19, 2020
@Fuzzbawls Fuzzbawls self-assigned this Dec 19, 2020
random-zebra and others added 16 commits December 20, 2020 21:31
(including delegated). This way, delegations are reflected under
"Pending balance" in the GUI (owner-side), before being included in a
block.

Github-Pull: PIVX-Project#2080
Rebased-From: d1f30ff
- no need to check "spendable" utxos (as we are not dealing with
watch-only).
- no need to catch `bad-txns-inputs-spent` exception with
sendrawtransaction (as we are not dealing with zPIVs anymore)
- double check that the double-spend is not on chain/mempool with
getrawtransaction

Github-Pull: PIVX-Project#2081
Rebased-From: 5546422
Verifier: (`CheckBlock` --> `IsBlockPayeeValid` -->
`IsBudgetPaymentBlock`) expects a payment to a budget even with the case
of N<20 active masternodes on the network, provided that all N voted on
the finalized budget (the threshold is N-1).

Also remove the extra call to `IsBudgetPaymentBlock` directly from
`FillBlockPayee` (it is already called by
`CBudgetManager::FillBlockPayee` --> `GetPayeeAndAmount`).

Github-Pull: PIVX-Project#2084
Rebased-From: 16455e0
We adopted a global `.h` header file extension convention from bitcoin,
this renames a number of header files coming from ZCash's sapling base
to conform to that convention.

Github-Pull: PIVX-Project#2087
Rebased-From: 47ce226
This command has been disabled since v4.1.0, don't bother registering it
 anymore so that the runtime error message isn't shown when using the
 `help` RPC command

Github-Pull: PIVX-Project#2088
Rebased-From: 2dd232d
>>> backports bitcoin/bitcoin@f0e7aa7

This prepares for a series of two additional commits which optimize
prevector performance.

Github-Pull: PIVX-Project#2083
Rebased-From: e6741d0
>>> based on bitcoin/bitcoin@e46be25
(updated with more recent changes)

In prevector.h, the code which like item_ptr(size()) apears in the loop.
Both item_ptr() and size() judge whether values are held directly or
indirectly, but in most cases it is sufficient to make that judgement
once outside the loop.

This PR adds 2 private function fill() which has the loop to initialize
by specified value (or iterator of the other prevector's element),
but don't call item_ptr() in their loop.
Other functions(assign(), constructor, operator=(), insert())
that has similar loop, call fill() instead of original loop.

Also, resize() was changed like fill(), but it calls the default
constructor for that element each time.

Github-Pull: PIVX-Project#2083
Rebased-From: c412d5f
It's now only referenced from the bench, so leave it there. This allows
us to
drop the associated includes as well.

Github-Pull: PIVX-Project#2083
Rebased-From: 4cb188e
Problem:
- IS_TRIVIALLY_CONSTRUCTIBLE macro does not work correctly resulting
  in `memset()` usage to set a non-trivial type to 0 when
  `nontrivial_t` is passed in from the tests.
- Warning reported by GCC when compiling with `--enable-werror`.

Solution:
- Use the standard algorithm `std::fill_n()` and let the compiler
  determine the optimal way of looping or using `memset()`.

Github-Pull: PIVX-Project#2083
Rebased-From: d856989
The count isn't updated on every widget show, only via the event.

Github-Pull: PIVX-Project#2075
Rebased-From: 164d6aa
Drop the "ed" suffix in the RPC scope, changes commands like
`getnewshieldedaddress` to `getnewshieldaddress`.

Also in blockchain informational calls; `shielded_pool_value` and
`shieldedsupply` become `shield_pool_value` and `shieldsupply`
respectively.

Github-Pull: PIVX-Project#2086
Rebased-From: b38077a
Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 9390f8e

@Fuzzbawls Fuzzbawls changed the title [WIP] [Backport] 5.0 backports [Backport] 5.0 backports Dec 21, 2020
Copy link

@furszy furszy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 9390f8e

@furszy furszy merged commit 6c8abbd into PIVX-Project:5.0 Dec 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants