ALL: Don't use EventRecorder at all when not compiled in
TEST: tests for common/huffman.h

A few tests for the Huffman decoder.

The encoding is the example from Wikipedia.
This could be improved by someone more knowledgeable by
generating one at runtime or using multiple encodings
which would each contain one edge case.
TEST: test cases for common/util.cpp
TEST: tests for common/rendermode.h

tests for parseRenderMode, renderMode2GUIO and getRenderModeCode.
TEST: reorganize test/common/util.h

Split a few test cases in two.
The resulting file is now better organized.
TEST: Thorough testing for common/hash-str.h

We test the various equal_to and hash functions therein.
TEST: removed fixed hashes in test/common/hash-str.h

The hash function does not necessarily have to conform to one specific algorithm as long as equals/differs is respected.
TEST: Whitespaces in test/common/
GRAPHICS: Be more robust with broken/unsupported thumbnail headers.

This fixes future issues like bug #3614654:
"ALL: ScummVM 1.5.0 can't read newer saved games".

There are a few behavior changes introduced with this commit:

- checkThumbnailHeader will now also report the presence of
  unsupported/broken (but skippable) headers.

- skipThumbnail will also try to skip the data for broken/unsupported
  thumbnail data.

- loadThumbnail will skip over broken/unsupported thumbnail data but still
  return 0 in this case.
Merge pull request #355 from lordhoto/skip-thumbnail

GRAPHICS: Be more robust with broken/unsupported thumbnail headers.
TSAGE: Extra comments for crane control scene
DEVTOOLS: Fix unimportant memory/resource leaks

CID 1022283, 1022284, 1022285
DEVTOOLS: Fix unimportant memory/resource leaks

CID 1022280, 1022281, 1022282
TSAGE: Fixes for R2R Crane Controls
TUCKER: Fix for bug #3614697 - "Spanish version uncompletable..."

This add support for the unhandled "mof" instruction present in the
Spanish version. This effectively skips it as a NOP/invalid, which
prevents it locking up the game at the end of the 2nd part.
TUCKER: Add support for skipping unhandled/invalid instructions.

This ensures that if any unhandled/invalid instructions are present,
they are skipped, rather than locking up engine execution.

This is a secondary fix for bug #3614697 - "Spanish version
uncompletable at end of 2nd part" and any other future bugs of this
type.
TSAGE: Fix for returning to landing bay from crane controls
RECORDER: Fix CID 1046886. Uninit class variables.
RECORDER: Fix CID 1046884. Uninitialized class variables.
RECORDER: Rename class variable in accoudance with our code style.