fix 2 compiler warnings

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

remove duplicate definition of SecureString in util.h

remove unused typedef in serialize.h

SecureString was moved to allocators.h in commit 6cb6d623

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

value [-Wparentheses]" in util.cpp

[-Wparentheses]" in net.cpp

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

Fix critical UI performance issue (#1154)

listsinceblock: rpc param blockid -> blockhash

Remove unused -DNOPCH

remove strange debug message from listsinceblock

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

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

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.

Address many more sign comparison warnings

This resolves signed/unsigned comparison warnings.

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.

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

Fixes several sign-comparison warnings.

Make GetDataDir return absolute paths

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.