NEVERHOOD: Remove NRect constructor

Otherwise, every global variable of type NRect requires a constructor to be
run, which can cause portability issue.
GUI: Use Domain::const_iterator in EventRecorder code when possible.
COMMON: Do not let ConfigManager::Domain inherit from StringMap.

Instead we provide wrapper functionality now. Intuitively a Domain is not a
simple StringMap.
COMMON: Rename ConfigFile to INIFile.

This clears up that 'ConfigFile' is actually a class handling only INI-files.
COMMON: Remove outdated comments on INIFile.

These very old comments claimed that ConfigManager uses INIFile. This is not
true however. Thus removing them should clear things up a little bit.
COMMON: Remove commented out include in config-manager.h.
LURE: Fix some formatting.
Merge pull request #373 from lordhoto/ini-file

Rename ConfigFile to INIFile and remove outdated comments
Merge pull request #369 from fingolfin/fix-neverhood

NEVERHOOD: Remove NRect constructor
WINTERMUTE: Avoid setting alpha-image when filename is ""
WINTERMUTE: Fix uninitialized scalar field (CID1060922)
MORTEVIELLE: Use more accurate frequency for music
MORTEVIELLE: Play ambiant sounds
MORTEVIELLE: some refactoring in ambiant sound code
MORTEVIELLE: Handle sounds
TINSEL: Fix DW2 assertion when quitting during cutscene

Call _bmv->FinishBMV() before setting g_pCurBgnd to NULL. Otherwise,
there will be an assertion if quitting during a cutscene while a
subtitle is being shown. (At least I think that's the condition.)

(Also, it seems to be a good idea to call it before EndScene(),
because even though there was no assertion, I got a lot of warnings
when I did it after.)
Merge branch 'master' of github.com:scummvm/scummvm
MORTEVIELLE: Some refactoring in noise sound code
MORTEVIELLE: Avoid loading sfx and noise files multiple times
MORTEVIELLE: Fix check of end of stream