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.

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.

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

Encapsulate RPC command dispatch in an array of CRPCCommand's

Merge pull request #1122 from dlitz/unsigned-char-fix

Unsigned char fix & fix undefined phexdigits[255]

Add missing includes. (Fix bulding under GCC 4.7)

Merge pull request #959 from rebroad/LoadBlockIndexKillable

Added ability to respond to signals during Block Loading stage.

Added ability to respond to signals during Block Loading stage.

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.

Merge pull request #1118 from sipa/addbreaks

Add missing breaks in optionmodel's switch case

Add missing breaks in optionmodel's switch case

Merge pull request #1114 from sipa/lesssync

Reduce sync frequency for blkindex.dat

Fix tests after recent refactors

Merge remote-tracking branch 'jgarzik/mempool'

Merge pull request #1117 from sipa/deadlockfix

Fix potential deadlock

Merge pull request #1112 from sipa/saneserial

Further reduce header dependencies

Further reduce header dependencies

This commit removes the dependency of serialize.h on PROTOCOL_VERSION,
and makes this parameter required instead of implicit. This is much saner,
as it makes the places where changing a version number can have an
influence obvious.

Move proto version to version.h.  Reduce header deps a bit more.

Move CWalletDB code to new walletdb module.

In addition to standard code separation, this change opens the door
to fixing several include inter-dependencies.

Remove headers.h

Merge pull request #1106 from jgarzik/sign-compare

Fix many sign-comparison warnings found in bitcoin codebase

Merge pull request #1115 from laanwj/2012_04_cleanupmisc

Delete unused, unreachable and commented code, add missing initializer

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.

main.cpp: replace tabs with spaces

Sometimes they sneak in through the 'vi' door

AlreadyHave(): only hold lock during mapTransactions access

Remove unused and unreachable code

Add forgotten initializer

Reduce sync frequency for blkindex.dat

Since auto-remove-db-logs was enabled, each time a CTxDB was closed
outside of the initial download window, it causes a checkpoint + log
cleanup. This is overkill, so reduce the sync frequency to once per
minute at most.

Merge pull request #1094 from jgarzik/already-have-locking

Locking fix for AlreadyHave()

Bump PROTOCOL_VERSION to 60001, thereby enabling BIP31

Merge pull request #1081 from jgarzik/pong

BIP 0031: pong message

Merge pull request #1092 from laanwj/2012_04_sendcoins_setlabelfix

When sending coins, set label when selecting address that already has a label

Merge pull request #1091 from Diapolo/GUI-BlockDL

revert to default OS theme for progressbar (fix #1071)

Fix misc. minor sign-comparison warnings

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

serialize.h: CAutoFile's read and write may take size_t nSize

Fix sign-compare warnings: netbase's Lookup* max-solutions may be unsigned

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.

CTxMemPool: add helper methods, to reduce global mempool.mapTx accesses

CTxMemPool: encapsulate AcceptToMemoryPool

CTxMemPool: encapsulate AddToMemoryPoolUnchecked(), RemoveFromMemoryPool(),
and nPooledTx

New class CTxMemPool, encapsulating TX memory pool data members

Merge pull request #1104 from laanwj/2012_04_clang

Enable and fix most compilation warnings

Merge pull request #1084 from dooglus/validate_secret

importprivkey doesn't check that it is given a private key

Merge pull request #1097 from laanwj/2012_04_runawayexception

Show a message box when runaway exception happens

Merge pull request #1100 from luke-jr/qrcode_errchk

Bugfix: Check that QRcode_encodeString didn't return NULL (error)

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

fix warnings: '&&' within '||' [-Wlogical-op-parentheses]

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]

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

enable all warnings except invalid-offsetof, sign-compare, unused-parameter

Bugfix: Check that QRcode_encodeString didn't return NULL (error)

Without this, any error will segfault Bitcoin-Qt

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.

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

Merge pull request #1093 from jgarzik/tx-opt

Database micro-optimization for "tx" network message

Merge pull request #1089 from laanwj/2012_04_translationupdate

Update translations for bitcoin core

Update translations for bitcoin core

- Move scripts/qt to share/qt, to clean up toplevel directories
- Update english ts file which is used to source messages for Transifex
- In extract_strings_qt.py use a glob *.h *.cpp, this is safe now that the Wx UI files are removed

Merge pull request #1096 from laanwj/2012_04_tick

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

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

Locking fix for AlreadyHave()

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

Also, reformat long lines to make the switch statement more readable.

Database micro-optimization for "tx" network message

Open database once per "tx" message, rather than multiple times,
in the case of orphan transaction presence.

As a side effect, a now-unused CTransaction::AcceptToMemoryPool()
variant is removed.

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

revert to default OS theme for progressbar / fix small glitches in bitcoingui.cpp

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

Merge pull request #1087 from sipa/fix_1086

Fix #1086: add /testnet to passed datadir

Fix #1086: add /testnet to passed datadir

Replace several network protocol version numbers with named constants
stored in version.h.

Also, a minor CAddress code reformat while we're in there, fixing
some incorrect indentation.

CBitcoinSecret::SetString() now calls IsValid() to make sure it was passed something with the correct version.

Merge pull request #1041 from gavinandresen/listtransactionsfix

Bug fix listtransactions from/count handling.

make text in about box selectable (fixes #1055)

Merge pull request #1078 from Diapolo/QR-Code

QR-Code code update

Fixes problems reported on forum: https://bitcointalk.org/index.php?topic=73695.msg839804#msg839804

enable wordWrap on lblQRCode / small code comment change

limit length of generated URI to 255 chars to prevent a DoS against the QR-Code dialog

updated to reflect pull-request suggestions / renamed some GUI elements

fixed amount part of URI in QR-Codes / removed (no label) string if we have NO label / coding style updates / removed an unused variable

Remove obsolete Win32 AffinityBugWorkaround()

Reference miner exists for testnet-in-a-box type situations, and as a
reference.  We don't care enough about highly optimized internal
mining to keep workarounds like this.

version.h: separate client, net sections.  Move more constants to this file.

* move PROTOCOL_VERSION to version.h

* move CLIENT_VERSION* to version.h, make available past cpp stage

* clearly separate client, network version portions of version.h

BIP 0031: pong message

Add a pong message that is sent in reply to a ping. It echoes back a nonce
field that is now added to the ping message. Send a nonce of zero in ping
messages.

Original author: Mike Hearn @ Google

Modified Mike's change to introduce a mild form of protocol documentation in
version.h.

Fix OSX build errors.

Merge pull request #855 from sje397/ToggleHide

Toggle UI hide

Merge pull request #1072 from sipa/boostpaths

Use filesystem::path instead of manual string tinkering