Load creature sounds when transforming

Closes #354
Default value for Avatar::loadSounds() param
Merge remote-tracking branch 'upstream/master' into bugfixes
NPC::loadSound(filename) was shadowing a member variable.
Merge pull request #377 from hean01/master

Second attempt for bugfixes
Merge pull request #379 from hean01/master

Fixes BUILD ERROR in previous merge
Merge remote-tracking branch 'upstream/master' into bugfixes

Conflicts:
	src/Avatar.cpp
Merge remote-tracking branch 'upstream/master' into bugfixes
Merge pull request #372 from dorkster/bugfixes

Load creature sounds when transforming
Merge remote-tracking branch 'mainline/master'
Fix #380.

This was caused by mechanically translating the assignments from structs
to pointers.
Looking at that part of the code makes me feel silly:
We are assigning a new AvatarStatBlock to the pointer and
then (without freeing) in the next line we are assigning another value
to the hero_stats pointer.

In the long term we should not copy so much stuff around when transforming
and untransforming, but rather be smart in statBlock() and return the
correct statBlock depending on transform state.
Merge pull request #381 from stefanbeller/master

Fix #380 and rename a parameter.
Revert "Merge pull request #378 from stefanbeller/master"

This reverts commit 5d1d4d657440984e260b7acb0d92a9f86fe478d3, reversing
changes made to 0f69b31ed167cc98c7f0d10c22de6202b262fb48.

So I run
  git revert --mainline 1 5d1d4d657440984e260b7acb0d92a9f86fe478d3
to purge all the changes I started on the StatBlock reorganisation.
I tested specially the transforming of the Avatar and it seems to work
again nicely.

Conflicts:
	src/Avatar.cpp
	src/GameStatePlay.cpp
	src/NPC.cpp
Merge pull request #382 from stefanbeller/revert

Revert "Merge pull request #378 from stefanbeller/master"