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/
RECORDER: Fix wrong argument order
GUI: Allow the event recorder dialog to be closed.

This is a regression from 6e4217e1ba1df9fd2ee4cd0d61151ed7dfc53021.

setResult does not automatically close the dialog. However,
Dialog::handleCommand will close the dialog when kCloseCmd is received.
GUI: Mark some intentional fall throughs in switches.

All of these are for handling kCloseCmd.
Merge pull request #323 from tobiatesan/test_for_common

Test for common
SWORD25: Replace Pluto with Tamed Pluto for increased compatibility

Still saves are incompatible but at least this now has enhanced
debugging capabilities.
SWORD25: Fix compilation on some 32-bit systems
CONFIGURE: Added more 64bit integers to probing
CONFIGURE: Use -Wno-long-long when we use -pedantic.
TSAGE: Bugfixes for R2R drive room, and work on scanner modal dialog
SCUMM: Better Player_Mac::durationToSamples

Uses the fact that 4*480*480 == 225 << 12, and the identity
(a*b)>>n == (a>>n)*b + ((a%(1<<n))*b)>>n (assuming non-overflowing math),
except the rhs uses smaller intermediate values and does not overflow(*).
Compared to the original code, this uses 1 fewer division and eliminates
the rounding error.

(*) Technical note: In some cases the right hand side of the above
identity still has possibilities of intermediate overflow, but only if
b > (1 << n), or if (b << n) overflows, neither of which are true here.
TSAGE: Fix stray "You have no use for that." message

This would happen in Ringworld when clicking on the slot in the
flycycle. I have compared the behaviour to the original, and I
can't see any obvious differences after my change. CID 1003728
TUCKER: Amend fix for bug #3614697 - "Spanish version uncompletable..."

The mof instruction needs to return 0 or 1 to continue execution, rather
than 2 which indicates to end table instruction execution. Using 0 as
this is most likely correct value.