Bumped version numbers to 0.4.0rc1

Merge pull request #486 from TheBlueMatt/translatefix

Update binary mos to latest translations.

Versions 0.3.20 THROUGH 0.3.23 have trouble with blockchain downloads; avoid them

Update binary mos to latest translations.

Stay connected to seed nodes; disconnecting causes problems if you are trying to make the initial blockchain download.

If compiled -DDEBUG_LOCKORDER and run with -debug, print out every mutex lock/unlock (helpful for debugging something-is-holding-a-mutex-too-long problems)

Do not try to download blockchain from 0.3.23 nodes

Give hard-coded seed nodes a random last-seen time, to randomize order they're tried.

Update the list of seednodes.

Nodes culled from MagicalTux's database of bitcoin nodes,
http://dump.bitcoin.it/misc/
by version and longevity, and tested for connectivity.

Fix bad merge: getaccountaddress was broken for new accounts

Fixed regression I introduced: wallets with lots of transactions were unusable in GUI.

Merge pull request #467 from gavinandresen/keypoolzero

Logic running with -keypool=0 was wrong (empty keys were being returned).

Merge branch 'code-cleanup' of git://github.com/muggenhor/bitcoin

obtain cs_wallet mutex to protect vchDefaultKey

Fix RPC call name in error message.

Logic running with -keypool=0 was wrong (empty keys were being returned). Fixes #445

Renames GetOrReuseKeyFromKeyPool to GetKeyFromPool, with fAllowReuse arg and bool result.

Merge pull request #470 from fabianhjr/master

Checkpoints

SetCrypted() obtains keystore lock, to be safe.

Fix typo ("you own security")

Merge pull request #480 from gavinandresen/deadlocks

Simplify mutex locking, fix deadlocks. Fixes issue #453

Fixed potential deadlocks in GUI code.
Also changed semantics of CWalletTx::GetTxTime(); now always returns the time the transaction was received by this node, not the average block time.
And added information about -DDEBUG_LOCKORDER to coding.txt.

Fix rpc-hanging deadlocks

Collapsed multiple wallet mutexes to a single cs_wallet, to avoid deadlocks with wallet methods that acquired locks in different order.
Also change master RPC call handler to acquire cs_main and cs_wallet locks before executing RPC calls; requiring each RPC call to acquire the right set of locks in the right order was too error-prone.

Highlight mis-matching locks

Merge pull request #463 from TheBlueMatt/encreadme

Encryption readme update and minor rpc.cpp fixes

Add reference python miner, in contrib/pyminer/

Merge pull request #475 from ovdeathiam/patch-1

Edited locale/pl/LC_MESSAGES/bitcoin.po via GitHub

Merge pull request #464 from TheBlueMatt/upnp1.6

Upgrade dependancies and tweak build process.

CHECKMULTISIG unit tests.

Add Gitian Build descriptors for Boost and wxWidgets.

Merge pull request #474 from xHire/master

Updated czech translation [only .po]

Mingw gitian build with deterministic bitcoin.exe by use of faketime

Mingw gitian with separate wxWidgets and boost

Linux gitian config with separate wxWidgets build

Upgrade dependancies and tweak build process.

* Upgrade to use miniupnpc 1.6
* Upgrade to wxWidgets 2.9.2
* Upgrade to Bost 1.47 for Win32 Builds

Edited locale/pl/LC_MESSAGES/bitcoin.po via GitHub

Move CInv to protocol.[ch]pp

This commit does *not* and should not modify *any* code, it only moves
it from net.h and splits it across protocol.cpp and protocol.hpp.

Signed-off-by: Giel van Schijndel <me@mortis.eu>

Move CAddress to protocol.[ch]pp

This commit does *not* and should not modify *any* code, it only moves
it from net.h and splits it across protocol.cpp and protocol.hpp.

Signed-off-by: Giel van Schijndel <me@mortis.eu>

Start moving protocol-specific code to protocol.[ch]pp

Move CMessageHeader from net.h to protocol.[ch]pp, with the
implementation in the .cpp compilation unit (compiling once is enough).

This commit does *not* and should not modify *any* code, it only moves
it from net.h and splits it across protocol.cpp and protocol.hpp.

Indentation changes aside the closest thing to a modification of code is
the addition of the 'TODO' comment (the execution of which requires code
modifications and thus doesn't belong in this commit).

Signed-off-by: Giel van Schijndel <me@mortis.eu>

Move func 'REF' from util.h to serialize.h

util.h doesn't use REF, serialize.h does, creating a dependency of
serialize.h on util.h, but util.h already depends on serialize.h.  To
resolve this circular dependency the function 'REF' has now been moved
closer to one of its two points of use.

Signed-off-by: Giel van Schijndel <me@mortis.eu>

Cleanup makefiles such that diffs to them are smaller

Signed-off-by: Giel van Schijndel <me@mortis.eu>

Make some global variables less-global (static)

Explicitly make these global variables less-global to reduce the maximum
scope of this global state.

In my experience global variables tend to be a major source of bugs. As
such the less accessible they are the less likely they are to be the
source of a bug.

Signed-off-by: Giel van Schijndel <me@mortis.eu>

Compile with DEBUG_LOCKORDER to detect inconsistent lock orderings that can cause deadlocks

Updated czech translation

Remove unused ScanMessageStart function

src/makefile.unix: remove -DFOURWAYSSE2

Spotted by Venkatesh Srinivas <me@endeavour.zapto.org>

Updated checkpoints, maybe Tx fee should be reduced to 0.0001 from 0.0005 and maximum minimum tx should be 0.0010.

Don't std::advance past beginning of transactions array.  Fixes #465

Add specific wallet encryption details to doc/README

Fix incorrect RPC error messages

Missed a 'password' should be 'passphrase'.

Merge pull request #458 from TheBlueMatt/copyright

Unify copyright notices.

Merge pull request #459 from jgarzik/char-msgstart

Use 'unsigned char' rather than 'char' for pchMessageStart.

Merge pull request #460 from jgarzik/make-tuple

Qualify make_tuple with boost:: namespace.

Use 'unsigned char' rather than 'char' for pchMessageStart.

Regarding https://bitcointalk.org/index.php?topic=28022.0

main.cpp has: "char pchMessageStart[4] = { 0xf9, 0xbe, 0xb4, 0xd9 };"
Per discussion on the thread linked, leaving the signedness of
pchMessageStart is unsafe for values > 0x80. This patch specifies
'unsigned char' in main.cpp and net.h.

Signed-off-by: Jeff Garzik <jgarzik@pobox.com>

Qualify make_tuple with boost:: namespace.

db.cpp has a number of uses of make_tuple and has 'using namespace
std' and 'using namespace boost'. Without qualifying make_tuple,
std::make_tuple is preferred, which is incorrect. This patch qualifies
make_tuple.

Signed-off-by: Jeff Garzik <jgarzik@exmulti.com>

Test for SO_NOSIGPIPE rather than assuming all BSDs support it.

Signed-off-by: Jeff Garzik <jgarzik@exmulti.com>

Do-nothing MapPort() ifndef USE_UPNP.  fixes #450

Unify copyright notices.

To a variation on:
// Copyright (c) 2009-2010 Satoshi Nakamoto
// Copyright (c) 2011 The Bitcoin developers

Cleanup test suite output to be more useful.

Add file for transaction tests.

Add a file for script tests

Add prototype for EvalScript() to script.h

This is needed for the script unit tests.

Fix testing setup

There were some problems with the existing testing setup:

 - Makefile rules for test-file compilation used CFLAGS instead of
   CXXFLAGS in makefile.unix

Add missing include to serialize.h

Add missing includes to net.h

Add missing include to script.h

Add missing includes to key.h

Merge pull request #443 from TheBlueMatt/dupin

Check for duplicate txins in CheckTransaction.

Merge branch 'tmp2'

Merge pull request #448 from sipa/cbitcoinaddress

avoid strAddress + validity checks

Merge pull request #446 from WakiMiko/upnp-desc

Identify as "Bitcoin + version number" when mapping UPnP port

Merge pull request #451 from TheBlueMatt/tornolisten

Don't listen if on TOR (resolves #441).

Don't listen if on TOR (resolves #441).

Make it clear that setting proxy requires restart to fully apply.

avoid strAddress + validity checks

Avoid references to addresses using strings, and use CBitcoinAddress
as much as possible. Also added some validity checks on addresses
entered using RPC.

Merge pull request #444 from Jeroenz0r/patch-2

Translation from "Open Bitcoin" to "Verstuur Bitcoins"

Identify as "Bitcoin + version number" when mapping UPnP port

Makes Bitcoin identify itself as "Bitcoin + version number" instead of
the nondescript "libminiupnpc" when forwarding a port via UPnP.

Merge pull request #437 from codler/patch-2

Comment "deprecated"

Translation from "Open Bitcoin" to "Verstuur Bitcoins"

Check for duplicate txins in CheckTransaction.

Comment "deprecated"

Merge pull request #436 from TheBlueMatt/master

Fix EncryptKeys crash introduced by a9ba471, identified by TD.

Fix EncryptKeys crash introduced by a9ba4710, identified by TD.

CAddrDB::LoadAddresses: properly initialize CAddress

Fixes issue #424

Merge pull request #435 from sipa/fixextract

Bugfix: don't overuse limited ExtractAddress

Bugfix: don't overuse limited ExtractAddress

ExtractAddress was called with the keystore as argument in RPC and
UI, limiting results to own keys. This caused empty "address"
fields.

Merge pull request #434 from laanwj/addkey_retval

make SetHash160 return a value (as specified in the function signature)

make SetHash160 return a value (as specified in the function signature)

Fix UNIX-specific thread handle leak.

Merge pull request #403 from sipa/cbitcoinaddress

keys indexed by address + introduced CBitcoinaddress

Merge pull request #428 from sipa/mintxfeefix

Fix for small change outputs

Fix for small change outputs

With the separation of CENT and MIN_TX_FEE, it is now reasonable
to create change outputs between 0.01 and 0.0005, as these are
spendable according to the policy, even though they require a fee
to be paid.

Also, when enough fee was already present, everything can go into
a change output, without further increasing the fee.

Merge pull request #423 from TheBlueMatt/mapalreadyaskedforfix

Actually use mapAlreadyAskedFor.

Actually use mapAlreadyAskedFor.

Previously, mapAlreadyAskedFor was read from, but never added to.
The original intent was to use mapAlreadyAskedFor to keep track
of the time an item was requested and "Each retry is 2 minutes
after the last".
This implements that intent.

Merge pull request #1 from AbrahamJewowich/cbitcoinaddress

Cbitcoinaddress

Fix bug with accessing vchData[0] when vchData is empty.
Fix typo in CBase58Data::CompareTo

Merge pull request #419 from mibe/translation-de

Updated German translation

Merge pull request #395 from mibe/issue-395

Error in Russian Translation

split off CBase58Data from CBitcoinAddress

Split off features unrelated to addresses from CBitcoinAddress to
CBase58Data, so they can be reused.

Use CBitcoinAddress instead of string/uint160

Instead of conversion functions between pubkey/uint160/address in
base58.h, have a fully fledged class CBitcoinAddress (CAddress was
already taken) to represent addresses.