NULL-check animation in Loot class's copy constructor

Fixes #947
Add missing defaults for inventory size.
MenuInventory: Replace "if( foo)" by "if (foo)"
MenuInventory: Fix potential memory access fault

When dropping vendor items on to a filled inventory slot, 'drag_prev_src'
is -1, so 'drag_prev_slot' will be calculated to be
inventory[-1].drag_prev_slot. However that value is never used.

As we don't know which memory we're accessing with inventory[-1]
(Well, some random memory before the inventory definition, which
currently luckily happens to be still member variables of MenuInventory),
we may get an memory access violation. As we're not using the computed
value, we're safe to omit calculating it in the first place.

Also see fe2dfbd18693f317 (2012-10-02, Fix crash when dropping vendor
items on to filled inventory slots)
Add missing default for stash
Widget: Move initialization of variables to parent ctor
Merge pull request #949 from dorkster/fixes

NULL-check animation in Loot class's copy constructor
Merge pull request #951 from stefanbeller/master

Fix potential bugs from coverity scan
Implement Rect class to replace SDL_Rect

Closes #950
Merge pull request #952 from dorkster/rect

Implement Rect class to replace SDL_Rect
Merge remote-tracking branch 'upstream/master'

Conflicts:
	src/UtilsDebug.cpp
	src/UtilsDebug.h
Use Rect class in SDL2RenderDevice and fix broken build
Use SDL2 by default when building with CMake