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.

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)

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.

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

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.

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

Use filesystem::path instead of manual string tinkering

Where possible, use boost::filesystem::path instead of std::string or
char* for filenames. This avoids a lot of manual string tinkering, in
favor of path::operator/.

GetDataDir is also reworked significantly, it now only keeps two cached
directory names (the network-specific data dir, and the root data dir),
which are decided through a parameter instead of pre-initialized global
variables.

Finally, remove the "upgrade from 0.1.5" case where a debug.log in the
current directory has to be removed.

Merge pull request #1083 from laanwj/2012_04_connectionicon

Make 4th bar of connection icon green

Remove path.make_preferred() calls, and fix compiler error in bitcoinrpc RE: boost::system

by popular request, make 4th bar of connection icon green

Merge pull request #1077 from sipa/buildinfo

Fix build.h dependencies

Fix build.h dependencies

For Qt builds, the build.h file is moved to build/build.h. For regular
builds, it is moved to obj/build.h. This allows the Qt build to be done
in a different directory than the source, and without interfering with
other builds.

Force regeneration of build.h in qmake builds

Merge pull request #1054 from sipa/buildinfo

Build identification strings

Merge pull request #1046 from laanwj/2012_04_rpcporterror

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

Build identification strings

All client version information is moved to version.cpp, which optionally
(-DHAVE_BUILD_INFO) includes build.h. build.h is automatically generated
on supporting platforms via contrib/genbuild.sh, using git describe.

The git export-subst attribute is used to put the commit id statically
in version.cpp inside generated archives, and this value is used if no
build.h is present.

The gitian descriptors are modified to use git archive instead of a
copy, to create the src/ directory in the output. This way,
src/src/version.cpp will contain the static commit id. To prevent
gitian builds from getting the "-dirty" marker in their git-describe
generated identifiers, no touching of files or running sed on the
makefile is performed anymore. This does not seem to influence
determinism.

Toggle main window hide on tray icon click

- converted openBictoinAction to toggleHideAction
- put GUIUtil functions into a namespace instead of a class
- put window-related functions together in optionsdialog

Reasoning:
- toggle is more typical behaviour
- it's more functional
- better UX

The typical issue with toggling visibility is that when a window
is obscured by other windows but in the 'shown' state, hiding it
isn't what you want. I've added an 'isObscured' function to GUIUtil
that checks several pixels in the window to see if they are visible
on the desktop so that an obscured but shown window can be raised.

Conflicts:

src/qt/guiutil.cpp
src/qt/guiutil.h

Merge pull request #1070 from cardpuncher/patch-1

Trivial: added comment lines in French & Turkish

Merge pull request #1052 from sipa/scopedlocks

Use scoped locks instead of CRITICAL_BLOCK

Added comment lines in French & Turkish which basically mean "Bitcoin, virtual P2P cryptocurrency".

Merge branch 'filesystem' of https://github.com/Diapolo/bitcoin

Do not report spurious deadlocks caused by TRY_LOCK

Support for parametrized locks in deadlock detector

Use scoped locks instead of CRITICAL_BLOCK

Merge pull request #987 from luke-jr/ipc_name_fix

Cleanup: Replace "URL" with "URI" where we aren't actually working with URLs

Bugfix: Replace "URL" with "URI" where we aren't actually working with URLs

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

Fixes issue #875

fixed small error in bitcoinrpc.cpp

updated util.cpp to use make_preferred()

updated bitcoinrpc.cpp to use make_preferred() and removed double inclusion of boost/filesystem.hpp

updated db.cpp to use make_preferred()

Merge pull request #1033 from sipa/wait

Condition variables instead of polling

Merge pull request #1051 from graingert/patch-1

Change sign message bitcoin address tooltip to "The address to sign the ...

Change sign message bitcoin address tooltip to "The address to sign the message with" Closes #1050

Merge pull request #1047 from luke-jr/strlcpy_attribute

Document strlcpy.h in assets-attribution.txt since it isn't MIT-licensed

Document strlcpy.h in assets-attribution.txt since it isn't MIT-licensed

Merge pull request #1045 from sipa/fastblocks2

Flush on log size instead of transaction count

Flush on log size instead of transaction count

Merge pull request #1043 from Diapolo/misc

removed obsolete boost workaround (boost ticket #4258)

Merge pull request #1042 from gavinandresen/use_ssl_cleanup

Remove USE_SSL #define

Merge pull request #1037 from gavinandresen/addmultisigenable

Enable addmultisigaddress RPC call for main network

Merge pull request #1036 from gavinandresen/pubsubcleanup

Remove half-implemented publish/subscribe system

Merge pull request #1035 from gavinandresen/osxbuild

Add ifdef RELEASE to makefile.osx as the compile-for-older-macs flags

Merge pull request #1039 from sipa/fix_955

Fix #955: Verify status of encrypt/decrypt calls to detect failed padding

removed obsolete boost workaround (boost ticket #4258)

Merge pull request #1038 from Diapolo/misc

small misc syntactic fixes

Remove USE_SSL #define

Bug fix listtransactions from/count handling.

Verify status of encrypt/decrypt calls to detect failed padding

removed an unneeded space in bitcoin-qt.pro

removed an unused / obsolete tag from bitcoin.qrc

Enable addmultisigaddress RPC call for main network

Remove half-implemented publish/subscribe system

Add ifdef RELEASE to makefile.osx as the compile-for-older-macs flags

Condition variable for outbound connection slots

Keep a global counter for nOutbound, protected with its own waitable
critical section, and wait when all outbound slots are filled, rather
than polling.

This removes the (on average) 1 second delay between a lost connection
and a new connection attempt, and may speed up shutdowns.

Locking system overhaul, add condition variables

This commit simplifies the locking system: CCriticalSection becomes a
simple typedef for boost::interprocess::interprocess_recursive_mutex,
and CCriticalBlock and CTryCriticalBlock are replaced by a templated
CMutexLock, which wraps boost::interprocess::scoped_lock.

By making the lock type a template parameter, some critical sections
can now be changed to non-recursive locks, which support waiting via
condition variables. These are implemented in CWaitableCriticalSection
and WAITABLE_CRITICAL_BLOCK.

CWaitableCriticalSection is a wrapper for a different Boost mutex,
which supports waiting/notification via condition variables. This
should enable us to remove much of the used polling code. Important
is that this mutex is not recursive, so functions that perform the
locking must not call eachother.

Because boost::interprocess::scoped_lock does not support assigning
and copying, I had to revert to the older CRITICAL_BLOCK macros that
use a nested for loop instead of a simple if.

Merge pull request #1019 from laanwj/2012_03_uirefactor

Streamline UI ↔ Core interface

Merge pull request #1025 from Diapolo/GUI-BlockDL

modified block DL progressbar to be more informative and precise

Move from noui.h / ui.h to one ui_interface.h with dummy implementation for the daemon.

Use a messagebox to display the error when -server is provided without providing a rpc password

qtui.h/noui.h interface cleanup

- rename wxMessageBox, remove redundant arguments to noui/qtui calls
- also, add flag to force blocking, modal dialog box for disk space warning etc
- clarify function naming
- no more special MessageBox needed from AppInit2, as window object is created before calling AppInit2

support RPC stop and encryptwallet with UI

move translation function _ to qtui.h/noui.h instead of util.h