Merge branch '0.4.x' into 0.5.x

Conflicts:
doc/release-process.txt
src/serialize.h

Fix build of testcases after commit 0f10b21719e1b0d9683a142f0a7105e65f095694

Bugfix: Fix various places where Bitcoin-Qt was being shutdown improperly

(Partial/merge of upstream 9247134eaba9a1d0fa74f22de238af1476663005, 1a3f0da9229a8e524d1010cdc8bd3b9da71fe529, and 3e343522226e6c249f5cb05436eec347b87c6361)

Don't call exit() in Shutdown() for Bitcoin-Qt (fixes a tray-icon issue)

(partial of 9247134eaba9a1d0fa74f22de238af1476663005)

Update wiki changelog at doc/release-process.txt

change initial Balance on overviewpage from "123.456 BTC" to "0 BTC" to not confuse users, which could see it before we init with the real wallet balance

removed ability to translate "0 BTC" and "123.456 BTC" as this is only used as preview in the Qt Designer anyway

(partial of 4295311da34ed8132351855f057decedfe434b44)

Fix broken config files

Do not select first address automatically in the address book

This contributed to an accidental send (#1384), and has no clear advantage, better to disable it.

Bump VERSION so we can differentiate between fixed 0.4.7rc2 and not-fixed 0.4.7rc1

Bugfix: Move IsStandard scriptSig size check out of IsPushOnly, since BIP16 verification uses the latter too

This caused clients to reject block #177618 since it has a P2SH transaction with over 200 bytes in scriptSig.

(Upstream commit: e679ec969c8b22c676ebb10bea1038f6c8f13b33)

fix an incorrect if-clause in net.cpp

Serialize access to debug.log stream

Acquire an exclusive, advisory lock before sending output to debug.log
and release it when we're done. This should avoid output from multiple
threads being interspersed in the log file.

We can't use CRITICAL_SECTION machinery for this because the debug log
is written during startup and shutdown when that machinery is not
available.

(Thanks to Gavin for pointing out the CRITICAL_SECTION problems based
on his earlier work in this area)

Correct blockchain size in contrib/debian.

* Updates package description to note that blockchain now takes
  2+ GB instead of 150+ MB.

(PARTIAL of 8f6111bb9cd598954e9634d9fe4500fcf5ae83de)

"USE_UPNP=-" is needed to remove UPnP support.

.gitignore: add test_bitcoin

Merge branch '0.4.x' into 0.5.x

Conflicts:
src/ui.cpp
src/ui.h
src/uibase.cpp
src/xpm/about.xpm

Prevent crashes due to missing or corrupted blk????.dat records

Prevent crashes due to missing or corrupted database records

Any problems seen during deserialization will throw an uncaught
exception, crashing the entire bitcoin process.  Properly return an
error instead, so that we may at least log the error and gracefully
shutdown other portions of the app.

Correct debug.log output to show correct function the debug is coming from.

Filter out whitespace and zero-width non-breaking spaces in validator

- Fixes issues with copy/pasting from web or html emails (#1325)

Update License in File Headers

I originally created a pull to replace the "COPYING" in crypter.cpp and
crypter.h, but it turned out that COPYING was actually the correct
file.

Update License in File Headers

I originally created a pull to replace the "COPYING" in crypter.cpp and
crypter.h, but it turned out that COPYING was actually the correct
file.

remove 2 ugly spaces from a message string

(PARTIAL of 966ae00)

Add build directory to .gitignore, so that it's not tracked.

Always check return values of TxnBegin() and TxnCommit()

PARTIAL, since d68dcf7 isn't backported (yet)

Hopefully final fix for the stuck blockchain issue

Immediately issue a "getblocks", instead of a "getdata" (which will
trigger the relevant "inv" to be sent anyway), and only do so when
the previous set of invs led us into a known and attached part of
the block tree.

remove string "TextLabel" from warningLabel, as this is unneeded and as such is a silly translation less to do :)

Properly escape " in strings when exporting CSV

remove 2 ugly spaces from a string used in translations

Bitcoin-Qt: Remove redundant tooltip on optional transaction fee. Fixes #1218

Bump version to 0.5.6

Merge branch '0.4.x' into 0.5.x

Conflicts:
contrib/Bitcoin.app/Contents/Info.plist
doc/README
doc/README_windows.txt
share/setup.nsi
src/serialize.h

Bump version to 0.4.7

Update/fix translations

Merge branch '0.4.x' into 0.5.x

Prevent stuck download: correct solution

Pull request #948 introduced a fix for nodes stuck on a long side branch
of the main chain. The fix was non-functional however, as the additional
getdata request was created in a first step of processing, but dropped
in a second step as it was considered redundant. This commits fixes it
by sending the request directly.

Prevent stuck block download in large reorganisations

In cases of very large reorganisations (hundreds of blocks), a situation
may appear where an 'inv' is sent as response to a 'getblocks', but the
last block mentioned in the inv is already known to the receiver node.
However, the supplying node uses a request for this last block as a
trigger to send the rest of the inv blocks. If it never comes, the block
chain download is stuck.

This commit makes the receiver node always request the last inv'ed block,
even if it is already known, to prevent this problem.

fix #952 by checking if we have a new address or an updated label

Hide UI immediately after leaving the main loop.

Prevents it from seeming to hang during shutdown if shutdown is triggered while the window is open.

Merge branch '0.4.x' into 0.5.x

Bugfix: %-12I64d is not valid and causes the parameter to be skipped, use %12"PRI64d" instead

Conflicts:

src/walletdb.cpp

remove HTML code around "Wallet" (displayed on overview page) and use Qt tags for font settings

Bugfix: Remove redundant duplicate transaction check

Merge branch '0.4.x' into 0.5.x

Check earlier for blocks with duplicate transactions. Fixes #1167

Check earlier for blocks with duplicate transactions. Fixes #1167

remove obsolete BackupWallet() entry in wallet.h

add bitcoin-qt.rc to OTHER_FILES (shown in Qt Creator)

fix compiler warning "suggest parentheses around assignment used as truth
value [-Wparentheses]" in util.cpp

fix DEPENDPATH in the project file, as json has no include sub-dir and src was in twice

remove unused typedef in serialize.h

fix compiler error in bitcoinrpc RE: boost::system

(Partial of 2232717cba9e9f06a01e8f37bcea4e79ee65205f)

Merge branch '0.4.x' into 0.5.x

Conflicts:
doc/build-msw.txt
src/makefile.linux-mingw
src/makefile.mingw

Windows build: compile against openssl 1.0.1b

Windows build: compile against openssl 1.0.1b

Bump version to 0.5.5

Merge branch '0.4.x' into 0.5.x

Conflicts:
contrib/Bitcoin.app/Contents/Info.plist
doc/README
doc/README_windows.txt
share/setup.nsi
src/protocol.h
src/serialize.h

Bump version to 0.4.6

listsinceblock: rpc param blockid -> blockhash

This is more consistent with the rest of the labeling seen
by the user when accessing the rpc commands.

remove strange debug message from listsinceblock

We should include netinet/in.h to use sockaddr_in (POSIX.1-2001)

We should include netinet/in.h to use sockaddr_in (POSIX.1-2001)

CBlock::WriteToDisk() properly checks ftell(3) for error return

Rather than storing ftell(3)'s return value -- a long -- in an
unsigned int, we store and check a properly typed temp.  Then, assured a
non-negative value, we store in nBlockPosRet.

Merge branch '0.4.x' into 0.5.x

Conflicts:
src/main.cpp

Add missing includes. (Fix bulding under GCC 4.7)

(Note: GCC 4.7 build NOT tested with backports -Luke)

Fix bugs on 'unsigned char' platforms.

In ISO C++, the signedness of 'char' is undefined.  On some platforms (e.g.
ARM), 'char' is an unsigned type, but some of the code relies on 'char' being
signed (as it is on x86).  This is indicated by compiler warnings like this:

 bignum.h: In constructor 'CBigNum::CBigNum(char)':
 bignum.h:81:59: warning: comparison is always true due to limited range of data type [-Wtype-limits]

 util.cpp: In function 'bool IsHex(const string&)':
 util.cpp:427:28: warning: comparison is always false due to limited range of data type [-Wtype-limits]

In particular, IsHex erroneously returned true regardless of the input
characters, as long as the length of the string was a positive multiple of 2.

Note: For testing, it's possible using GCC to force char to be unsigned by
adding the -funsigned-char parameter to xCXXFLAGS.

Fix phexdigits[255] is undefined.

Add missing breaks in optionmodel's switch case

Fix potential deadlock

Conflict:
* cs_main in ProcessMessages() (before calling ProcessMessages)
* cs_vSend in CNode::BeginMessage
versus:
* cs_vSend in ThreadMessageHandler2 (before calling SendMessages)
* cs_main in SendMessages

Even though cs_vSend is a try_lock, if it succeeds simultaneously with
the locking of cs_main in ProcessMessages(), it could cause a deadlock.

Fix misc. minor sign-comparison warnings

CNode's nHeaderStart may be negative, so change its type (PARTIAL)

Fix loop index var types, fixing many minor sign comparison warnings

foo.size() typically returns an unsigned integral type; make loop variables
match those types' signedness.

The string class returns string::npos, when find() fails.

Noticed when sign-comparison warnings were enabled.

Add forgotten initializer

AlreadyHave(): only hold lock during mapTransactions access

Locking fix for AlreadyHave()

Access to mapTransactions[] must be guarded by cs_mapTransactions lock.

Set label when selecting an address that already has a label. Fixes #1080.

Merge branch '0.5.0.x' into 0.5.x

Merge branch '0.4.x' into 0.5.0.x

Conflicts:
src/keystore.h

fix warnings: suggest explicit braces to avoid ambiguous 'else' [-Wparentheses]

fix warnings: delete called on 'XX' that has virtual functions but non-virtual destructor [-Wdelete-non-virtual-dtor]

fix warnings: unused variable 'XX' [-Wunused-variable]

fix warnings: enumeration values 'XX' not handled in switch [-Wswitch-enum]

fix warnings: 'XX' defined as a struct here but previously declared as a class [-Wmismatched-tags]

fix warnings: array subscript is of type 'char' [-Wchar-subscripts]

fix warnings: array subscript is of type 'char' [-Wchar-subscripts]

work around issue in boost::program_options that prevents from compiling in clang

Show a message box when runaway exception happens

This is more clear to users than when the program simply disappears (usually during initialization). It still logs the message to the console and debug log as well.

Merge commit 'e962c7f' into 0.5.0.x

Bugfix: nTotalBlocks wasn't in 0.5.0, so need to replace it with equivalent function call in backport

Add symlink to scripts/qt/make_windows_icon.sh from old file name, just in case

Rename make_windows_icon.py to .sh as it is a shell script (fixes #1099)

Do not show green tick unless all known blocks are downloaded (fixes #921)

Add missing tooltip and key shortcut in settings dialog (#1088 without line break part)

Bug fix listtransactions from/count handling.

Show error message instead of exception crash when unable to bind RPC port

Fixes issue #875

Merge branch '0.5.0.x' into 0.5.x

Bugfix: Windows lacks sleep(), so need to use Sleep() from util.h

Merge branch '0.5.0.x' into 0.5.x

Conflicts:
src/qt/notificator.h