Sonar critical violations fixed

Some pending violations related to language syntax mainly.

Only tests that failed is the below, which fails even before the changes were applied. Otherwise build and test went fine.

=== performing get test ===
Average get time including startup, bootstrap and shutdown, took 0.045 us average per key
May 04, 2013 10:14:46 PM com.higherfrequencytrading.chronicle.tcp.InProcessChronicleSink readNextExcerpt
INFO: Lost connection to localhost/127.0.0.1:12348 retrying java.nio.channels.AsynchronousCloseException
Tests run: 5, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 20.164 sec <<< FAILURE!

Results :

Failed tests:   testOverTCP(com.higherfrequencytrading.chronicle.impl.InProcessChronicleTest): expected:<17869> but was:<83405>

Tests in error: 
  testOverTcp(com.higherfrequencytrading.chronicle.datamodel.MapWrapperTest): Address already in use

Tests run: 32, Failures: 1, Errors: 1, Skipped: 3