Rename Serialized*Size -> SerializedSprout*Size

Rename SpendingKeyMap -> SproutSpendingKeyMap

Pass SaplingPaymentAddress to store through the CKeyStore

Add CWallet::AddCryptedSaplingSpendingKey() hook

To be filled in when we implement Sapling persistence to disk.

Remove outdated comment

Check for unencrypted Sapling keys in CCryptoKeyStore::SetCrypted()

Add Sapling decryption check to CCryptoKeyStore::Unlock()

Rename CryptedSpendingKeyMap -> CryptedSproutSpendingKeyMap

Rename DecryptSpendingKey -> DecryptSproutSpendingKey

Add Sapling keys to CCryptoKeyStore::EncryptKeys

Add Sapling have/get sk crypter overrides

Auto merge of #3340 - Eirik0:rename-merkle-typedefs, r=str4d

Rename merkle tree typedefs to include Sprout

This is to be consistent with the naming convention we have been using to distinguish Sprout/Sapling.

Auto merge of #3439 - ebfull:fix-ecc, r=ebfull

Don't call ECC_Start/ECC_Stop outside the test harness.

Make changes to gtest ECC behavior suggested by @str4d.

Don't call ECC_Start/ECC_Stop outside the test harness.

Auto merge of #3246 - str4d:checkpoint-304600, r=bitcartel

Add some more checkpoints, up to the 1.1.0 release

Auto merge of #3429 - str4d:3217-sapling-address-creation, r=str4d

Add Sapling support to z_getnewaddress and z_listaddresses

Closes #3217.

test: Update rpc_wallet_z_importexport to account for Sapling changes

test: Fix permissions of wallet_addresses

Rename test objects to include sprout or sapling

Rename Witnesses to include sprout or sapling

Rename Merkle Trees to include sprout or sapling

test: Another assert in wallet_zkeys_tests.store_and_load_sapling_zkeys

test: Deduplicate logic in wallet_addresses RPC test

Auto merge of #3417 - str4d:sapling-tx-builder, r=ebfull

TransactionBuilder for Sapling and transparent transactions

This PR includes zcash/librustzcash#24, and will create a testnet chain split once merged into master (and once a Sapling transaction is mined).

Relocate ECC_Start() to avoid test failures.

Formatting

Auto merge of #3430 - ebfull:check-valuebalance, r=ebfull

Ensure sum of valueBalance and all vpub_new's does not exceed MAX_MONEY

HT to @daira for noticing that we don't check this in `CheckTransactionWithoutProofVerification`. We actually do check it in `GetShieldedValueIn` (by throwing an exception) but these exceptions are not handled by `CheckTxInputs`, which may cause a remote DoS.

Not exploitable assuming our proving system is secure, but definitely worthy of a fix.

Move `extern params` to beginning of `test_checktransaction`.

Ensure sum of valueBalance and all vpub_new's does not exceed MAX_MONEY inside of CheckTransactionWithoutProofVerification.

Fix bug in return value.

Update librustzcash and sapling-crypto.

Auto merge of #3336 - arcalinea:expiryheight-createrawtransaction, r=bitcartel

Take expiryheight as param to createrawtransaction

Addresses #3333

Add tests for expiryheight parameter of RPC createrawtransaction.

Fix block hash for checkpoint at height 270000

Add Sapling support to z_getnewaddress and z_listaddresses

Raise the 90-character limit on Bech32 encodings

Regtest addresses are 91 characters, and ZIP 32's Bech32 encoding will
be significantly longer.

Implement CKeyStore::GetSaplingPaymentAddresses()

Rename xsk to expsk

xsk will be used for ZIP 32 extended spending keys, so renaming here to
reduce confusion.

TransactionBuilder: Make fee configurable

TransactionBuilder: Add change output to transaction

Auto merge of #3410 - LarryRuane:3399-peer-ban, r=bitcartel

don't ban peers when loading pre-overwinter blocks

Addresses #3399. If we see an invalid transaction (from our peers who are up to date, post-overwinter) during initial block download, don't invoke DoS (logging messages, blocking these peers). Instead, simply ignore the transaction.

TransactionBuilder: Add support for transparent inputs and outputs

Auto merge of #3402 - mdr0id:3401_fix_auto_senescence, r=bitcartel

Fix minor references to auto-senescence in code

Fix references

Auto merge of #3392 - arcalinea:sapling_z_importexport_keys, r=bitcartel

Add Sapling support to z_importkey, z_exportkey

Add Sapling support to `z_importkey` and `z_exportkey`

Clean up help messages for RPC createrawtransaction.

Also return error if expiryheight used when Overwinter not active.

Take expiryheight as param to createrawtransaction

Refactor into visitors and throw errors for invalid key or address.

Add Sapling to rpc_wallet_z_importexport test

Add Sapling support to z_importkey

Add Sapling support to z_exportkey

Fix minor references to auto-senescence in code

Update typo in rc-notes-1.0.13

Fix typo in rc-notes-1.0.13

Fix typo in rc-notes-1.0.13

Fix typo in rc-notes-1.1.0

Fix typo in rc-notes-1.1.0

Fix typo in make-release

Revert prior release note term to original auto-senescence

test: Move ECC_Start() call into src/gtest/main.cpp

Formatting

TransactionBuilder: Check that all anchors in a transaction are identical

This reduces the amount of information that is leaked by the choice of anchor.
In future we will make a protocol change to enforce that all inputs use the
same anchor.

don't ban peers when loading pre-overwinter blocks

Auto merge of #3416 - charlieok:3415_fix_double_slash_error, r=bitcartel

Remove extra slash from lockfile path

Issue: https://github.com/zcash/zcash/issues/3415

After this change I can successfully run `fetch-params` in an msys2 environment.

Auto merge of #3353 - Eirik0:3062-cache-sapling-witnesses, r=bitcartel

Cache Sapling witnesses in the wallet

Closes #3062

I have not update the tests in test_wallet.cpp. Also, there are several other methods in the wallet that have to do with witnesses and note data which will need to be updated, but this PR focuses on IncrementNoteWitnesses and DecrementNoteWitnesses.

Clean up for rebase: rename mapNoteData to mapSproutNoteData.

Only check for a valid Sapling anchor after Sapling activation.

Before Sapling activates, hashFinalSaplingRoot is 0x0, which is
not a valid anchor.  After Sapling activates, the empty root value
of 0x3e49... is a valid anchor.

Add parameter for version in GetValidReceive

Add getter and setter for sapling note data and update tests

Remove extra indentation

Rename sprout specific methods

Increment sapling note witnesses

Pass sapling merkle tree when incrementing witnesses

Extract method for incrementing witness heights

Extract methods for incrementing witnesses

Extract method for copying previous witnesses

Clear sapling witness cache

Decrement sapling note witnesses

Add sapling note data and map

Add out point for sapling note data

Consolidate for loops

Remove redundant curly braces

Rename note data to include sprout

TransactionBuilder with support for creating Sapling-only transactions

Remove extra slash from lockfile path

Expose note position in IncrementalMerkleWitness

Alter SaplingNote::nullifier() to take a SaplingFullViewingKey

This means the API will work if you only have a SaplingExtendedSpendingKey,
as will be the case with ZIP 32.

Auto merge of #3391 - bitcartel:3061_sapling_note_encryption, r=ebfull

Sapling note plaintext (encryption and decryption)

Add encryption and decryption of SaplingNotePlaintext and SaplingOutgoingPlaintext classes.

This is part of #3061 to add Sapling note functionality.

Auto merge of #3408 - LarryRuane:2929-python-test-nondeterministic, r=bitcartel

Add call to sync_all() after (z_sendmany, wait)

Closes #2929. Add missing call to `self.sync_all()` after sending coins (and waiting for the transaction to complete). This matches what is done in other python test scripts.

Rename AttemptSaplingEncDecryptionUsingFullViewingKey and use function overloading.

Implement Sapling note decryption using full viewing key.

Add call to sync_all() after (z_sendmany, wait)

Auto merge of #3350 - str4d:rpc-reorg, r=bitcartel

RPC code refactor

Cherry-picked from the following upstream PRs:

- bitcoin/bitcoin#7307
- bitcoin/bitcoin#7348
- bitcoin/bitcoin#7766

rpc: Register calls where they are defined

Split out methods to every module, apart from 'help' and 'stop' which
are implemented in rpcserver.cpp itself.

- This makes it easier to add or remove RPC commands - no longer everything that includes
    rpcserver.h has to be rebuilt when there's a change there.
- Cleans up `rpc/server.h` by getting rid of the huge cluttered list of function definitions.
- Removes most of the bitcoin-specific code from rpcserver.cpp and .h.

Continues #7307 for the non-wallet.

rpc: update inline comments to refer to new file paths

move rpc* to rpc/

[RPC, Wallet] Move RPC dispatch table registration to wallet/ code

Allow extending the rpc dispatch table by appending commands when server is not running.

Auto merge of #3376 - LarryRuane:2728-help-message-experimental, r=bitcartel

add extra help how to enable experimental features

Closes #2728. When the user attempts to execute a disabled experimental RPC, print some nice text explaining how to enable the RPC (either command-line or adding lines to zcash.conf).

Auto merge of #3371 - kozyilmaz:clang, r=str4d

Add -Wl,-pie linker option for macOS and use it instead of -pie

fixes linker error (see below) due to "-pie" usage, replaced it with "-Wl,-pie" for macOS platform
```
clang: error: argument unused during compilation: '-pie' [-Werror,-Wunused-command-line-argument]
```

Auto merge of #3243 - pejvan:patch-1, r=bitcartel

Update README.md

Added default options to build zcash with as many parallel compilation units as there are cpu cores.
This makes compiling zcash multiple times faster.

Auto merge of #3354 - leto:absurd_fee_bug, r=bitcartel

Fix absurd fee bug reported in #3281, with tests

This was ported from the Hush `absurd_fee_bug` PR: https://github.com/MyHush/hush/pull/136

Tests executed with `./qa/pull-tester/rpc-tests.sh wallet` and pass on Ubuntu 16.04. Some tests were added for previously untested behavior as well.

Closes #3281.

Minor update to address nits in review.

Auto merge of #3263 - str4d:ibd-upstream-changes, r=bitcartel

InitialBlockDownload upstream changes

Cherry-picked from the following upstream PRs:

- bitcoin/bitcoin#7208
- bitcoin/bitcoin#8007
- bitcoin/bitcoin#9053
  - Excluding second commit (requires bitcoin/bitcoin#8865)
- bitcoin/bitcoin#10388

Update and fix per review comments, the test for absurd fee.

Auto merge of #3156 - Eirik0:2935-change-indicator, r=bitcartel

Add change indicator

This adds to the json returned when calling z_listreceivedbyaddress and z_listuspent an additional field entitiled 'change' which will be either true or false depending on whether or not the note was change from another transaction.

Update README.md

updated to use parenthesis as suggested by @bitcartel

add extra help how to enable experimental features