Another removal of boost dependency as well as fixing up some
`-Wthread-safety-analysis` issues.

Code to avoid calling Perfmon too often is only needed when perfmon is actually going to get called.
This is not intended to make any functional difference in the addition of entropy to the random pool.

No need to define it for every CAddress, as it's memory only anyway.

[Refactoring] Moved and removed some stuff
-Removed duplicated arith uint files
-Removed unused variables
-Move keystore impls to .cpp instead of .h
-Removed useless function in key.cpp
[Crypto] fix bip38 compilation for latest libsecp256k1
[Compilation] Change compilation and some code to use libsec instead of sslcrypto
[Crypto] Update keys to comply with latest secp256k1 lib

With C++11 these functions are not needed anymore and gracefully
replaced

The zerocoin database should be wiped on reindex. If leftover information from a fork is left in the db it can lead to potential validation issues in the future and isolated forks.

rebase from upstream. Windows doesn't initialize CService properly
during startup before the masternode.conf file is read.

Custom boost locations not using pkg_config can result in a header
include not being found when compiling the libzerocoin library. This
quick fix ensures that the `BOOST_CPPFLAGS` are explicitely included.

Tor Icon is displayed when the client is initialized with a successful
tor service connection.

Icon is hidden otherwise.

Instead of using boost::lexical_cast we can just use std::to_string

Use proper include ordering and only include the needed QStyle header.

Also use proper define in header file

Locked balances were being double counted in the total and available
values. This fixes the issue once and for all.

fixes PIVX-Project#654

(cherry picked from commit bad068a)

There were surprisingly many `#ifdef` fallbacks for Qt 4.

Removing them simplifies maintenance, as well as adding new GUI
functionality.

(cherry picked from commit 907f73b)

Fix an edge case where autocombine would get stuck if available
UTXOs are low in value compared to autocombine threshold

This file has not been used in quite a while, nuke it from orbit.

Prevent file access collisions and possible file corruption by refusing
to overwrite the current wallet in use.

(cherry picked from commit 908c1d7)

(rebased from bitcoin/bitcoin@fc6a9f2)

This function is never used, and never will be; Nuke it from orbit.

verifychain only accepts a single (optional) argument instead of two,
and we need to capture that argument properly.

Properly return an error when attempting to use a negative timeout. Also
 sanitize the timeout so as to not cause a value error with libevent on
 MacOS.

Even though this command is all but deprecated, it's help output and
example were incorrect and deserving of an update.

amount is the 3rd paramater, and is required. minconf is the 4th
paramater, and for all intents and purposes is a "dummy" paramater since
 it is no longer considered in the functional logic.

When calling `getreceivedbyaddress` with a non-wallet address, return a
proper error code/message instead of `0`

check-doc and logprint-scanner were still using python2 conventions.
This updates the two scripts to use pyhon3 instead and be consistent
with the rest of our python utility scripts.

Also, use a more widely accessible bash path in lint-whitespace.sh

Used primarialy (for now) in node identification for upcoming regtest
scripting.

Simple addition to the JSON output; mainly used in upcoming functional
test updates.

Simple consistency addition. This mimmics the output from `getinfo`.

- Fix git's rev-list call
- Add proper copyright headers from upstream
- Check that the script is in the git tree

This moves CScheduler and threadGroup to a static declaration in init
.cpp so as to avoid a potential shutdown deadlock where both are freed
before the network message handler thread has been completely released.

use c++11 for loops instead of BOOST_FOREACH

- include memory system header for std pointers
- remove unused MutexName
- declare single-argument constructors as explicit

When running the client with the `-disablewallet` option, the staking
and automint icons should always be hidden. This adds a check to see if
`walletFrame` has been set before showing/updating these two icons.

Add Checks for local endian and byteswap support.

Add env prefix to bash scripting

caea417 Update QT to 5.9.7 (cevap)

Tree-SHA512: 34197e7b92711f3f0392b071de58900592ab8bb2ed3e50cd853cf2da56ba2f6d9b3ee57e4d01f3c2cdce4710ebe0c079bba434263b9af65141363086b9213025

…ith gitian

f6c1d4a Add check symbols for gitian-linux.yml (cevap)
4a3d3ea Use pivx in config lines (cevap)
321f0e0 Update linux descriptor (cevap)
4e0244b Update symbol check script (cevap)
c251de5 Update glibc_compat.cpp with risc (cevap)
fb0bdae Gitian: add risc build (cevap)

Tree-SHA512: f91fa1da9c48e3ccb87cb1ddd825d353a15e439b778bd95eb53f22b8884ecb1ddda954a506553dbb8322e931d1b8b34211e1dc7e4b431eafa107d81bdd8e6faa

Addresses https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2019-6250

Signed-off-by: cevap <dev@i2pmail.org>

Github-Pull: PIVX-Project#858
Rebased-From: 21f5078

Break out a new job to do the basic linting that check-dock.py and
logprint-scanner.py did.

Also add new scripts to check the sanity of
any git subtrees and to check that new pull requests don't contain any
trailing whitespace.

Also, remove the outdated slack notification callback

Also for libsecp256k1 now that it has been updated