VIDEO: Wrap 8-bit smacker audio properly

The accumulator 'bases' is 16-bit but when used in 8-bit audio we need to
wrap as if 'bases' is 8-bit. Clipping on output is no longer required.

This fixes noise in The Neverhood's in-game "making of" videos,
particularly the section called "Construction, powertools & painting",
hash 0x21080009.

The intro video in The Neverhood is 16-bit audio and was not affected by
this bug. No other videos or games have been tested.
TONY: Fix potential memory leak

Unfortunately, I'm not sure where and when this function is called.
But this should be correct. CID 1003575
TONY: Fix memory leak on save game failure (CID 1003579)

Though shouldn't it also alert the user to the failure?
SCUMM: Verify sample width in Digital iMUSE callback. CID 1002112

Verify that 'bits' really is one of 8, 12 or 16 before decoding
the data. It's probably always the case (unless the data files are
damaged) but if it isn't we'll either try to queue NULL to the
audio stream, or queue the same buffer more than once, or free the
buffer more than once. All of which are bad, though Coverity only
noticed the last of these cases.
HOPKINS: Fix 2 glitches reported by greencis in win95 demos
LURE: Use correct deallocator, CID 1004041

The memory was allocated with Memory::alloc(), so it should be
freed with Memory::dealloc(), not delete.
BUILD: install tony.dat
HOPKINS: Fix glitch occurring in the intro of the Linux Demo.

Thanks greencis for reporting it
TINSEL: Revert the incorrect flagging of DW2 entries as CD versions

This reverts the changes done to the DW2 entries in commit d4a354c1.

We only distinguish between floppy and CD versions if a game had both a
CD and a floppy version released. DW2 was only released as a CD version,
so the extra flagging of the DW2 entries as CD was redundant, and caused
confusion regarding previous saved games to users that readded the DW2
detection entries after this change
ALL: Fix typo (succesful -> successful)

Thanks to 'onlyjob' on pull request #337 for pointing out an instance of this.
BUILD: GNU Hurd & kFreeBSD support

From: Petr Salinger <Petr.Salinger@seznam.cz>
Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=711320
Bug-ScummVM: https://sourceforge.net/tracker/?func=detail&atid=418820&aid=3614268&group_id=37116
COMMON: Add depth/flat parameters to addSubDirectoryMatching.
MOHAWK: Use addSubDirectoryMatching for LB paths.
WINTERMUTE: Fix for unnecessary "WARNING: getChildren() failed for path:"

FSNode::getChildren() was called assuming that it returns 0 iff success.
Actually, it's the other way around.
WINTERMUTE: Prevent warning about duplicate files when autodetecting.

Removed a call to BaseFileManager::registerPackages() in
WintermuteEngine::getGameInfo(), as the constructor for BaseFileManager
already calls registerPackages() once; another call should thus be
superfluous.
Merge pull request #336 from madmoose/smacker-8-bit-audio-fix

VIDEO: Wrap 8-bit smacker audio properly
COMMON: Fix regression in SineTable creation.

This is a regression from f4ba8a6485b097a8ef1e2004d1af127243f379f1. The
commit replaced the static cosine and sine tables with dynamically created
ones. In the process of that a copy&paste error happened which made the sine
table use the layout of the cosine table. This commit now changes the
dynamically created sine tables to conform to the layout of the previous
static tables.
COMMON: Save memory by allocating only required entries in Cosine-/SineTable.

The tables only contain (2^bitPrecision)/2 entries. The code allocated twice
as many entries previously.
COMMON: Try to document the SineTable/CosineTable table entries.
COMMON: Improve comment in CosineTable::CosineTable.

This commit changes the comment to use the same variable names as we do in the
code. Furthermore, it also makes the comment a bit easier to grasp.