Move CBlockIndex, CChain and related code out of main

Decouple CChain from mapBlockIndex

Merge pull request #4754

e088d65 Separate script/sign (jtimon)
9294a4b Separate CScriptCompressor (jtimon)
c4408a6 Separate script/standard (jtimon)
da03e6e Separate script/interpreter (jtimon)
cbd22a5 Move CScript class and dependencies to script/script (jtimon)
86dbeea Rename script.h/.cpp to scriptutils.h/.cpp (plus remove duplicated includes) (jtimon)

Rebased-by: Pieter Wuille

Separate script/sign

Separate CScriptCompressor

Separate script/standard

Separate script/interpreter

Move CScript class and dependencies to script/script

Rename script.h/.cpp to scriptutils.h/.cpp (plus remove duplicated includes)

Merge pull request #4865

8e44f2e Clarify 'compressed nor uncompressed' error message (Peter Todd)

Merge pull request #4712

80daee0 [Qt] Call checkBalanceChanged() periodically instead for every updated transaction (Cozz Lovan)

Merge pull request #4697

da2ede2 [Wallet] Improve ReorderTransactions(..) (Cozz Lovan)

Merge pull request #4862

45a4baf Add testnet DNS seed of Andreas Schildbach. (Andreas Schildbach)

Clarify 'compressed nor uncompressed' error message

Add testnet DNS seed of Andreas Schildbach.

It runs sipas crawler, but rather than using its custom nameserver implementation it serves a generated zonefile via bind9. The zone always contains 25 IPv4 and 25 IPv6 peers. FWIW, the zone is secured using DNSSEC.

Merge pull request #4851

41ef558 univalue: make spaceStr thread-safe (Wladimir J. van der Laan)

Merge pull request #4839

346193b Cleanup messy error messages (R E Broadley)

Cleanup messy error messages

Merge pull request #4849

a264e44 remove dup include of foreach.hpp in script.cpp (Philip Kaufmann)

Merge pull request #4836 from morcos/fix-make_change

Fix make_change to not create half-satoshis

Fix make_change to not create half-satoshis

univalue: make spaceStr thread-safe

Simply add spaces to the existing string instead of using a
temporary.

Fixes #4756.

Improve readability of CAddrInfo::IsTerrible

- Replaced 86400 with 24*60*60
- Remove references to specific timespans in comments

Github-Pull: #4724

Merge pull request #4850

1ffb99b [Qt] copyright, style and indentation cleanup of Qt tests (Philip Kaufmann)

[Qt] copyright, style and indentation cleanup of Qt tests

Merge pull request #4843

81061ff travis: add osx build (Cory Fields)
e4cf9f8 depends: fix typo in source downloading. Regression from c897b1e732. (Cory Fields)

remove dup include of foreach.hpp in script.cpp

Merge pull request #4838

1e4f87f Use memcmp for uint256 equality/inequality (Pieter Wuille)
8a41e1e Use boost::unordered_map for mapBlockIndex (Pieter Wuille)
145d5be Introduce BlockMap type for mapBlockIndex (Pieter Wuille)
a0dbe43 checkpoints.cpp depends on main, it can use mapBlockIndex directly (Pieter Wuille)

travis: add osx build

depends: fix typo in source downloading. Regression from c897b1e732.

Some sources are renamed after download, since the filenames don't play nice
with (for example) gitian. This fixes the rename.

Needed for OSX build as it renames a file.

Merge pull request #4783

fbe0fca [Qt] minor watch-only changes (Philip Kaufmann)

Merge pull request #4762

399cdbc contrib/linearize: Add feature to set file's timestamp based on block header time. (Jeff Garzik)
8f5a423 contrib/linearize: split block files based on year-month, not just year (Jeff Garzik)
75400a2 contrib/linearize: Guarantee that output is generated in-order (Jeff Garzik)

contrib/linearize: Add feature to set file's timestamp based on block header time.

Merge pull request #4833

bbda402 net: Remove MilliSleep from StopNode (Wladimir J. van der Laan)

Merge pull request #4790

9189f5f remove useless millisleep (phantomcircuit)

Merge pull request #4842

fb51e28 Added XML syntax highlighting (himynameismartin)

Added XML syntax highlighting

Use memcmp for uint256 equality/inequality

Use boost::unordered_map for mapBlockIndex

Introduce BlockMap type for mapBlockIndex

checkpoints.cpp depends on main, it can use mapBlockIndex directly

Merge pull request #4808

3f6540a Rename IMPLEMENT_SERIALIZE to ADD_SERIALIZE_METHODS (Pieter Wuille)
47eb765 Serializer simplifications after IMPLEMENT_SERIALIZE overhaul (Pieter Wuille)

Merge pull request #4820

910526d Use OR of respective block flags for masks (Pieter Wuille)

Merge pull request #4671

550d4fa Remove DNS Seeds run by entities which were never well-established. (Matt Corallo)

net: Remove MilliSleep from StopNode

I don't understand why it would be there in the first place. This looks
like voodoo, not programming.

Remove DNS Seeds run by entities which were never well-established.

remove useless millisleep

reduces time to service requests improving performance

Use OR of respective block flags for masks

Merge pull request #4812

53efb09 Discover some missing includes (jtimon)
8d5e510 Remove unused function StackString() and class CCoins; (jtimon)

Merge pull request #4810

c0f5d4a Fixing Compiler Error C2466 (ENikS)

Rename IMPLEMENT_SERIALIZE to ADD_SERIALIZE_METHODS

Discover some missing includes

Remove unused function StackString() and class CCoins;

Fixing Compiler Error C2466

Serializer simplifications after IMPLEMENT_SERIALIZE overhaul

Merge pull request #4737

31e9a83 Use CSizeComputer to avoid counting sizes in SerializationOp (Pieter Wuille)
84881f8 rework overhauled serialization methods to non-static (Kamil Domanski)
5d96b4a remove fields of ser_streamplaceholder (Kamil Domanski)
3d796f8 overhaul serialization code (Kamil Domanski)

Merge pull request #4779

093303a add missing header end comments (Philip Kaufmann)

Merge pull request #4767

b144a74 depends: bump miniupnpc to 1.9.20140701. (Cory Fields)
f628127 depends: bump openssl to 1.0.1i (Cory Fields)
9f7f504 build: add -DMINIUPNP_STATICLIB for new version (Cory Fields)

Merge pull request #4780

4d04492 add missing copyright headers (Philip Kaufmann)

Merge pull request #4776

6f5d33b Update translations after update script improvements (Wladimir J. van der Laan)
da59f28 Add deeper XML checking to update-translation script (Wladimir J. van der Laan)

Update translations after update script improvements

Add deeper XML checking to update-translation script

- Catch problems such as mismatched formatting characters. Remove
  messages that can give problems at runtime.

- Also remove unfinished/untranslated messages, they just take up space
  in the ts and waste parsing time.

Fixes #4774.

Merge pull request #4768

2e28031 Perform CVerifyDB on pcoinsdbview instead of pcoinsTip (Wladimir J. van der Laan)

Use CSizeComputer to avoid counting sizes in SerializationOp

rework overhauled serialization methods to non-static

Thanks to Pieter Wuille for most of the work on this commit.
I did not fixup the overhaul commit, because a rebase conflicted
with "remove fields of ser_streamplaceholder".
I prefer not to risk making a mistake while resolving it.

remove fields of ser_streamplaceholder

The nType and nVersion fields of stream objects are never accessed
from outside the class (or perhaps from the inside too, I haven't checked).
Thus no need to have them in a placeholder, whose only purpose is to
fill the "Stream" template parameter in serialization implementation.

overhaul serialization code

The implementation of each class' serialization/deserialization is no longer
passed within a macro. The implementation now lies within a template of form:

template <typename T, typename Stream, typename Operation>
inline static size_t SerializationOp(T thisPtr, Stream& s, Operation ser_action, int nType, int nVersion) {
    size_t nSerSize = 0;
    /* CODE */
    return nSerSize;
}

In cases when codepath should depend on whether or not we are just deserializing
(old fGetSize, fWrite, fRead flags) an additional clause can be used:
bool fRead = boost::is_same<Operation, CSerActionUnserialize>();

The IMPLEMENT_SERIALIZE macro will now be a freestanding clause added within
class' body (similiar to Qt's Q_OBJECT) to implement GetSerializeSize,
Serialize and Unserialize. These are now wrappers around
the "SerializationOp" template.

changed field types in some structures to equivalent unambiguous types

Conflicts:
src/core.cpp

Rebased-By: Wladimir J. van der Laan
Github-Pull: #4180

Merge pull request #4789

bac5586 Replace weird characters by normal spaces (Pieter Wuille)

Merge pull request #4781

187115c cleanup include of assert.h (Philip Kaufmann)

Replace weird characters by normal spaces

Merge pull request #4778

Merge pull request #4788

missing include boost/algorithm/string/replace.hpp

Merge pull request #4599

Move CMerkleTx to wallet.cpp/h

It is only used by the wallet so it has no place in main.

rpc: Compute number of confirmations of a block from block height

Currently this uses a CMerkleTx, but that makes no sense as we
have the CBlockIndex available. As noted by @jgarzik.

build: Remove message about Ubuntu 13.10 when no boost sleep implementation found

It's only confusing people into thinking that they should mess with
boost versions, which should not be necessary to get bitcoind to work.

If there is a bug in the build system with autodetecting boost it needs
to be solved not worked around.

Merge pull request #4777

8bdd287 Fix a few "Uninitialized scalar field" warnings (Wladimir J. van der Laan)

[Qt] minor watch-only changes

- use watch-only, not watchonly
- add back a tooltip hint when hovering addresses and attach
  "(watch-only)" at the end

cleanup include of assert.h

add missing copyright headers

add missing header end comments

- ensures a consistent usage in header files
- also add a blank line after the copyright header where missing
- also remove orphan new-lines at the end of some files

Merge pull request #4772

87d9819 fix comments ExtractAddress() -> ExtractDestination() (jtimon)

Merge pull request #4766

5a0823a Add travis build:passing/failing indicator. (randy-waterhouse)

Fix a few "Uninitialized scalar field" warnings

Fix a few warnings reported by Coverity.
None of these is critical, but making sure that class fields are
initialized can avoid heisenbugs.

Add travis build:passing/failing indicator.

qt/splashscreen: #include version.h

Needed to build breakage reported by Arnavion on IRC:
qt/splashscreen.cpp: In constructor 'SplashScreen::SplashScreen(const QPixmap&, Qt::WindowFlags, bool)':
qt/splashscreen.cpp:33:98: error: 'FormatFullVersion' was not declared in this scope

fix comments ExtractAddress() -> ExtractDestination()

Fix build with DEBUG_LOCKORDER

Fixes #4771

Merge pull request #4377

654871d replace ComputeMinWork with CheckMinWork (jtimon)
b343c1a Move CBlockIndex::GetBlockWork() to pow::GetProofIncrement(nBits) (jtimon)
c2c02f3 Move UpdateTime to pow (jtimon)

Merge pull request #4764

92bb6f2 Bypass reloading blocks from disk (Pieter Wuille)

Revert "Add a getutxos command to the p2p protocol. It allows querying of the UTXO set"

This reverts commit da2ec100f3681176f60dec6dc675fc64147ade3a.

Merge pull request #4752

df7565d depends: add sensible download timeout/retry values (Cory Fields)
c897b1e depends: add a fallback path in case package sources go missing (Cory Fields)

depends: add sensible download timeout/retry values

Perform CVerifyDB on pcoinsdbview instead of pcoinsTip

Bypassing the main coins cache allows more thorough checking with the same
memory budget.

This has no effect on performance because everything ends up in the child
cache created by VerifyDB itself.

It has bugged me ever since #4675, which effectively reduced the
number of checked blocks to reduce peak memory usage.

- Pass the coinsview to use as argument to VerifyDB

- This also avoids that the first `pcoinsTip->Flush()` after VerifyDB
  writes a large slew of unchanged coin records back to the database.

depends: bump miniupnpc to 1.9.20140701.

This is the forward-port of a9c6eef915ee264ccf4169d5e8769a6be1fa101a

depends: bump openssl to 1.0.1i

This is the forward-port of bba01750226745d6666d587cabe57c321fde0875.

build: add -DMINIUPNP_STATICLIB for new version

libminiupnpc changed their required static define to the much more sane
"MINIUPNP_STATICLIB". Sadly, they don't respect the old "STATICLIB" for
back-compat. Define them both since the old one didn't seem to be conflicting
anywhere.

Also go ahead and split out the cppflags so that they can be applied only where
they're needed. This will help us to build dll's from our libs without having
their import/export declspecs poisoned.

[Qt] Call checkBalanceChanged() periodically instead for every updated transaction