removing a completely unneeded function
removing another completely pointless function
removed unused var
removed more warning and some pointless vars
missed a function def
Fix compile warnings printing size_t

Original fix in 8b9566bd was enough to suppress the warning when
building the 32bit version of the Windows agent but the 64bit builds
still complain. An extra layer of encapsulation was required.
Merge pull request #360 from awiddersheim/fix_win32_warning

Fix compile warnings printing size_t
[logcollector] fix compiler warnings
[logcollector] fix clang analyzer warning: Null pointer passed as an argument to a 'nonnull' parameter
[logcollector] remove duplicated statement
[logcollector] fix CID 28448
[logcollector] fix CID 28447
[logcollector] fix CID 28519
remove strange code, relating to logcollector
[tests] set timeout for OS_GetHost() tests to 10 seconds; thanks to @awiddersheim
better file handling on update:

always overwrite internal_options.conf
only write local_internal_options.conf if it not exists
only write client.keys if it not exists
Merge pull request #363 from cgzones/unit_test_timeout

[tests] set timeout for OS_GetHost() tests to 10 seconds
Merge pull request #364 from cgzones/install_config

better file handling on update
Merge pull request #361 from cgzones/logcollector

logcollector
[syscheck] fix compiler warnings