Merge pull request #1205 from laanwj/2012_05_granular_ui_notifications

Finer-grained UI updates, move UI interface to boost::signals

Make testcases build, prevent windows symbol collision

Process address book updates incrementally

- No longer invalidates selection model, thus retains selection on address book changes
- Fixes selection of new address when added

Convert UI interface to boost::signals2.

- Signals now go directly from the core to WalletModel/ClientModel.
  - WalletModel subscribes to signals on CWallet: Prepares for multi-wallet support, by no longer assuming an implicit global wallet.
- Gets rid of noui.cpp, the few lines that were left are merged into init.cpp
- Rename wxXXX message flags to MF_XXX, to make them UI indifferent.
- ThreadSafeMessageBox no longer returns the value `4` which was never used, converted to void.

Fine-grained UI updates

Gets rid of `MainFrameRepaint` in favor of specific update functions that tell the UI exactly what changed.

This improves the efficiency of various handlers. Also fixes problems with mined transactions not showing up until restart.

The following notifications were added:

- `NotifyBlocksChanged`: Block chain changed
- `NotifyKeyStoreStatusChanged`: Wallet status (encrypted, locked) changed.
- `NotifyAddressBookChanged`: Address book entry changed.
- `NotifyTransactionChanged`: Wallet transaction added, removed or updated.
- `NotifyNumConnectionsChanged`: Number of connections changed.
- `NotifyAlertChanged`: New, updated or cancelled alert. As this finally makes it possible for the UI to know when a new alert arrived, it can be shown as OS notification.

These notifications could also be useful for RPC clients. However, currently, they are ignored in bitcoind (in noui.cpp).

Also brings back polling with timer for numBlocks in ClientModel. This value updates so frequently during initial download that the number of signals clogs the UI thread and causes heavy CPU usage. And after initial block download, the value changes so rarely that a delay of half a second until the UI updates is unnoticable.

Merge pull request #1323 from Diapolo/string_fixes

translation updates / string updates

Merge pull request #1353 from Diapolo/RPC_OpenDebugLog_Icon

GUI: add an icon for Debug logfile -> Open in the RPC console

Merge pull request #1302 from laanwj/2012_05_utilstrings

Get rid of snprintf (except one) with fixed buffers, shorten code

Merge pull request #1360 from jgarzik/datasync

Default to DB_TXN_WRITE_NOSYNC for all transactional operations

Merge pull request #1358 from luke-jr/shared_lockchk

Shared code for wallet lock help and check

Merge pull request #1296 from Diapolo/CheckDiskSpace

make CheckDiskSpace() use 50 * 1024 * 1024 Bytes

Merge pull request #1289 from Diapolo/Bitcoin_as_noun

change strings to Bitcoin (uppercase), where it is used as a noun and up...

Merge pull request #1362 from laanwj/2012_05_debugwindowicon2

Replace debug window icon with LGPL-licensed one (fixes #1348)

Replace debug window icon with LGPL-licensed one (fixes #1348)

Default to DB_TXN_WRITE_NOSYNC for all transactional operations

* This is safer than DB_TXN_NOSYNC, and does not appear to impact
  performance.
* Applying this to the dbenv is necessary to avoid many fdatasync(2)
  calls on db 5.x
* We carefully and thoroughly flush databases upon shutdown and
  other important events already.

Shared code for wallet lock help and check

change strings to Bitcoin (uppercase), where it is used as a noun and update strings to use "Qt" (and not qt or QT) / update initialisation of notificator to use qApp->applicationName() instead of a static string

Merge pull request #1350 from jgarzik/del-deprecated

JSON-RPC: remove 'getblocknumber' deprecated RPC

Use boost::thread locking instead of interprocess

add an icon for Debug logfile -> Open in the RPC console / add a missing comment in rpcconsole.h

Merge pull request #1346 from laanwj/2012_05_overviewpage_restore

Restore overview page (put transactions back within frame)

Restore overviewpage (put transactions back within frame)

- Also, change "(out of sync)" to only red, instead of red and bold, which a bit more subtle
- Move label stylesheets to xml

JSON-RPC: remove 'getblocknumber' deprecated RPC

RPC 'getblockcount' should be used instead.

Make orphan logging more verbose, displaying mapOrphanTransactions.size()

Old log message:
storing orphan tx df2244f6bc

New log message:
storing orphan tx df2244f6bc (mapsz 51)

Also, trim a few trailing whitespace in main.cpp.

Merge pull request #1340 from rebroad/DBFlushDurationReport

Report how long DBFlush took.

Revert "use _strnicmp (ISO C++) instead of deprecated strnicmp (POSIX) and add missing "#define strncasecmp _strnicmp" in init.cpp"

This reverts commit f4ac41806af5766199a7d526a7becbcb8a0f5ab3.

Reason: breaks build.

Merge pull request #1345 from Diapolo/strnicmp_Win32

Win32: use _strnicmp (ISO C++) instead of deprecated strnicmp (POSIX)

Report how long DBFlush took.

use _strnicmp (ISO C++) instead of deprecated strnicmp (POSIX) and add missing "#define strncasecmp _strnicmp" in init.cpp

Merge pull request #1314 from Diapolo/Wallet_Transaction_state_Overviewpage

GUI: add 2 labels to the overviewpage that display Wallet and Transaction status

add 2 labels to the overviewpage that display Wallet and Transaction status (obsolete or current) / cleanup overviewpage XML ui-file

Merge pull request #1336 from laanwj/2012_05_consoleicon

Add icon for debug window

Add icon for debug window

Merge pull request #1335 from rebroad/IgnoreBuildDir

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

Merge pull request #1334 from rebroad/Exiting2Exited

Corrected grammar. As per Principle Of Least Surprise.

Merge pull request #1333 from rebroad/AddBinSh

Add /bin/sh to bitcoin-qt.pro - as some filesystems don't have the execute flag.

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

Corrected grammar. As per Principle Of Least Surprise.

Add /bin/sh to bitcoin-qt.pro - as some filesystems don't have the execute flag.

Merge pull request #1331 from roques/trivia

Fix typo.  libarcode => libqrcode

Merge pull request #1332 from Diapolo/fix_RenameOver_FileCommit_Win32

fix RenameOver() and FileCommit() functions, to not generate compilation errors

fix RenameOver() and FileCommit() functions, to not generate compilation errors

Merge pull request #1295 from jgarzik/txn-retval

[FIX] Always check return values of TxnBegin() and TxnCommit()

Merge pull request #1316 from jgarzik/dead-code

Remove dead code: CTxDB::EraseBlockIndex(), CBlockIndex::EraseBlockFromDisk()

Merge pull request #1198 from jgarzik/addrman

Replace BDB-managed addr.dat with bitcoin-managed peers.dat

Fix typo.  libarcode => libqrcode

allow translation of "options" used in the --help message / split translation of RPC console welcome message and remove the need to take care of "<br>" / remove some spaces in strings and misc other stuff related to translations

CAddrDB: Replace BDB-managed addr.dat with internally managed peers.dat

Add new utility functions FileCommit(), RenameOver()

Merge pull request #1319 from Diapolo/add_new_languages

add 4 new translations bg, el_GR, fr and pt_PT (based on transifex fetch from today)

Merge pull request #1324 from laanwj/2012_05_missing_qobject

Add missing Q_OBJECT in bitcoin.cpp

Add missing Q_OBJECT in bitcoin.cpp

Fixes translating HelpMessageBox strings.

Merge pull request #1321 from laanwj/2012_05_updatemaintrans

Update bitcoinstrings.cpp and bitcoin_en.ts

Update bitcoinstrings.cpp and bitcoin_en.ts

Merge pull request #1315 from sipa/unstuck

Hopefully final fix for the stuck blockchain issue

add 4 new translations bg, el_GR, fr and pt_PT (based on transifex fetch from today)

Remove dead code: CTxDB::EraseBlockIndex(), CBlockIndex::EraseBlockFromDisk()

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.

Merge pull request #1308 from Diapolo/polish_verifymessage

GUI: adapt user-experience from messagepage for verifymessagedialog

adapt user-experience from messagepage / move placeholderTexts from XML to source to avoid a problem with Qt < 4.7 / add eventFilter for address field to select text when clicking in / add Clear All button / rework strings

Merge pull request #1307 from Diapolo/fix_OpenLogFile

GUI fix: DebugLog file opens twice after clicking "Open" in RPC Console Information tab

Merge pull request #1262 from Diapolo/no_double_timestamps

no more double timestamps in debug.log

fix DebugLog file opens twice after clicking "Open" in RPC Console Information tab

Merge pull request #906 from sje397/ValidateMessage

Add a menu option and dialog to verify a signed message

Merge pull request #1301 from laanwj/2012_05_rpcscrolltoend

RPC console: scroll to the end when user enters a command

Get rid of snprintf (except one) with fixed buffers, shorten code

- Use strprintf or vstrprintf instead of snprintf

Merge pull request #1299 from sipa/ipv6fixes

A few more IPv6-related improvements

RPC console: scroll to the end when user enters a command

- Ensures that the command and reply is visible

Do not listen on blocked networks

Allow binding to the any address, and report failure

-onlynet instead of -blocknet

make CheckDiskSpace() use a global static const uint64 for checking required disk space and remove 2 ugly spaces from a message string

Merge pull request #1291 from sipa/canonames

Use getnameinfo() to get canonical IPv6 addresses

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

Use getnameinfo() to get canonical IPv6 addresses

Really do not consider blocked networks local

Merge pull request #1283 from laanwj/2012_05_prevent_tooltip_infiniteloop

Prevent tooltip filter from ever causing infinite loops

Merge pull request #1288 from Diapolo/sendcoinsdialog_strings

update some strings used as warning messages in sendcoinsdialog.cpp

Merge pull request #1284 from laanwj/2012_05_helpmessage

Move help message out of AppInit2

Merge pull request #1287 from sipa/noirc6

Only encode IPv4 addresses in IRC nicks

update some strings used as warning messages in sendcoinsdialog.cpp

Move help message out of AppInit2

- Solves #1278, attempts to address #1049
- Removes \t's from help message that are removed afterwards anyway
- Moves UI-specific command-line options help to UI code
- Moves "-detachdb" out of #ifdef USE_UPNP

release-process.txt: Add a note to check the download link destinations.

Do not consider blocked networks local

Only encode IPv4 addresses in IRC nicks

Merge pull request #1285 from sipa/initerror

Improve error reporting at startup

ensure that no double timestamps show up in the debug.log, by removing manual timestamps from the source (now only -logtimestamps parameter adds timestamps to debug.log)

Enforce -nodiscover better

Merge pull request #1178 from laanwj/2012_05_move_startonsystemstartup

Move GetStartOnSystemStartup and SetStartOnSystemStartup to GUI code

Improve error reporting at startup

Move GetStartOnSystemStartup and SetStartOnSystemStartup to GUI code

Merge pull request #1270 from laanwj/2012_05_overviewpage2

When a transaction is clicked on overview page, focus it on history page

Merge pull request #1274 from Diapolo/QR_Code

rename "QR-Code Dialog" to "QR Code Dialog" in qrcodedialog.ui as this spelling is used all over the code

Merge pull request #1275 from Diapolo/askpassphrasedialog

remove string "TextLabel" from warningLabel, as this is unneeded and as ...

Prevent tooltip filter from ever causing infinite loops

Merge pull request #1268 from Diapolo/transactiondialog

change size of transactiondialog, to allow display of transaction-id wit...

Merge pull request #1277 from Diapolo/IPv6_IPv4_strings

use "IPv6" and "IPv4" in strings as these are the official spellings

Merge pull request #1272 from sipa/ipv6fixes

A few IPv6 fixes

Fix warning about uninitialized value

Only reported when using -flto.

Take -port into account when resolving -bind's