Fixed non-sensical error message

Previously trying to create a multisig address that required less than
one signature would output something like the following:

"wrong number of keys(got 1, need at least 0)"

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

Merge branch '0.6.0.x' into 0.6.x

Conflicts:
bitcoin-qt.pro
contrib/gitian-descriptors/gitian-win32.yml
doc/README
doc/README_windows.txt
share/qt/make_windows_icon.sh
share/setup.nsi
src/bitcoinrpc.cpp
src/main.cpp
src/main.h
src/makefile.linux-mingw
src/net.cpp
src/net.h
src/netbase.h
src/noui.h
src/qt/addressbookpage.cpp
src/qt/bitcoin.cpp
src/qt/bitcoingui.cpp
src/qt/bitcoingui.h
src/qt/bitcoinstrings.cpp
src/qt/guiutil.cpp
src/qt/guiutil.h
src/qt/optionsmodel.cpp
src/qt/qrcodedialog.cpp
src/qt/qtipcserver.cpp
src/script.cpp
src/serialize.h
src/ui_interface.h
src/uint256.h

Merge branch '0.5.x' into 0.6.0.x

Conflicts:
src/main.cpp

Bugfix: Remove redundant duplicate transaction check

Merge branch '0.6.0.x' of gitorious.org:+bitcoin-stable-developers/bitcoin/bitcoind-stable into 0.6.0.x

Merge branch '0.5.x' into 0.6.0.x

Conflicts:
src/main.cpp
src/serialize.h

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

Bump version numbers for 0.6.1rc2

Merge pull request #1188 from Diapolo/rem_BackupWallet_wallet_h

remove obsolete BackupWallet() entry in wallet.h

remove obsolete BackupWallet() entry in wallet.h

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

Merge pull request #1179 from Diapolo/project-file

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

Merge pull request #1171 from Diapolo/fix_compiler_warn

fix 2 compiler warnings

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

Merge pull request #1176 from Diapolo/project-file

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

Merge pull request #1177 from laanwj/2012_05_remove_duplicate_securestring

remove duplicate definition of SecureString in util.h

Merge pull request #1172 from Diapolo/serialize_h_rem_typedef

remove unused typedef in serialize.h

remove duplicate definition of SecureString in util.h

SecureString was moved to allocators.h in commit 6cb6d623479c5dd42d91de7a4d391078d0800e54

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

Merge pull request #1173 from Diapolo/init_cpp_comment_upd

remove references to bitcoin-wx from comment in init.cpp

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

fix compiler warning "suggest explicit braces to avoid ambiguous "else"
[-Wparentheses]" in net.cpp

remove references to bitcoin-wx from comment in init.cpp

remove unused typedef in serialize.h

Check earlier for blocks with duplicate transactions. Fixes #1167

change button tooltip on sign message page for copy to clipboard as it was missleading

Merge pull request #1139 from Diapolo/messagepage

change button tooltip on sign message page for copy to clipboard...

Merge pull request #1163 from laanwj/2012_04_uiperformanceissue

Fix critical UI performance issue (#1154)

Fix critical UI performance issue (#1154)

Merge branch '0.5.x' into 0.6.0.x

fix compiler error in bitcoinrpc RE: boost::system

(Partial of 2232717cba9e9f06a01e8f37bcea4e79ee65205f)

Bump win32.deps version number for new openssl

Merge branch '0.5.x' into 0.6.0.x

Conflicts:
contrib/gitian-descriptors/gitian-win32.yml

Windows build: compile against openssl 1.0.1b

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

Update openssl version

Bump win32.deps version number for new openssl

Windows build: compile against openssl 1.0.1b

Merge branch '0.5.x' into 0.6.0.x

Conflicts:
bitcoin-qt.pro
doc/README
doc/README_windows.txt
share/setup.nsi
src/serialize.h

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

Windows build: compile against openssl 1.0.1b

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)

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

Bump version numbers for 0.6.1rc1

Merge pull request #1151 from freewil/listsinceblock-blockhash

listsinceblock: rpc param blockid -> blockhash

Merge pull request #1150 from gavinandresen/NOPCH

Remove unused -DNOPCH

Merge pull request #1152 from freewil/remove-strange-debug

remove strange debug message from listsinceblock

Merge pull request #1146 from drizztbsd/posix-include

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

remove strange debug message from listsinceblock

listsinceblock: rpc param blockid -> blockhash

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

Define TEST_DATA_DIR so unit tests can be run from any current working directory

Undo part of c2e8c8ac to fix issue#1148

Merge pull request #1119 from sipa/fastshutdown

Make lsn_reset ("detach databases") optional and off by default.

Remove unused -DNOPCH

Make lsn_reset ("detach databases") optional and off by default.

Add an option -detachdb (and entry in OptionDialog), without which no
lsn_reset is called on addr.dat and blkindex.dat. That means these
files cannot be moved to a new environment, but shutdown can be
significantly faster. The wallet file is always lsn_reset'ed.

-detachdb corresponds to the old behaviour, though it is off by
default now to speed up shutdowns.

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.

Test ScriptSigArgsExpected() for error, before accumulating return value

Merge pull request #1140 from jgarzik/sign-compare

Address many more sign comparison warnings

Change signed->unsigned at 3 code sites

This resolves signed/unsigned comparison warnings.

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.

Add casts for unavoidable signed/unsigned comparisons

At these code sites, it is preferable to cast rather than change
a variable's type.

Test ScriptSigArgsExpected() for error, before accumulating return value

SigOp and orphan-tx constants and counts are always unsigned.

Fixes several sign-comparison warnings.

Merge pull request #1133 from sipa/abspath

Make GetDataDir return absolute paths

Prefer 'unsigned int' for loop index variables tested against ::size()

C++ STL ::size() generally returns unsigned, which implies that "int idx"
style of loop variable will generate a signed-vs-unsigned comparison warning
when testing the loop exit condition "idx < blah.size()"

Update areas of the bitcoin code where loop variables may be more properly and
correctly defined as unsigned.

change button tooltip on sign message page for copy to clipboard as it was missleading

Make GetDataDir return absolute paths

Merge branch '0.5.x' into 0.6.0.x

Conflicts:
src/main.h
src/net.cpp
src/util.cpp

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 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.

Fix phexdigits[255] is undefined.

Merge pull request #1124 from sipa/rpcobj3

extension of #1103: encapsulate mapCommands in CRPCTable

Merge pull request #1131 from laanwj/2012_04_hexstr

Integrate @JoelKatz's optimized ToHex (#562) into current HexStr function

Merge pull request #1129 from laanwj/2012_04_opcodes

Add explicit numeric constant value for all opcodes

Merge pull request #1121 from gavinandresen/scripttest

Data-driven script evaluation unit tests (rebased to fix compiler warning).

Data-drive script evaluation unit tests.

Merge pull request #1126 from drizztbsd/master

Fix build with gcc 4.7

Integrate @JoelKatz's optimized ToHex (#562) into current HexStr function

Remove no-longer used UI hints in bitcoin core

The Qt UI has its own associated structures for temporary transaction state / cache.

change type of various bare chars to bool that are only used as bool (and never serialized)

Add explicit numeric constant value for all opcodes

- Easier for debugging (what opcode was 0x... again?)
- Clarifies that the opcodes are set in stone in the protocol, and signals that it is impossible to insert opcodes in between.

Expose CRPCTable via bitcoinrpc.h for testing

Encapsulate mapCommands in class CRPCTable