[xml] 'action' type setting should not require 'id' attribute
remove unused variable
fix uninitialized variable
[vtb] - fix - use the correct bit for signaling the hw decoder to drop a frame. This is the bit which is visible in the now public ios 8 headers of vtb. I doubt that apple ever change this till ios4. If they did - we are screwed on older runtimes. At least on ios6 this bit is valid for sure.
Fixes: check results of memory allocation functions
Fixes: check for NULL pointers before dereference
Fixes: don't use uninitialized vars
Fixes: checks for buffer overrun
Fixes: wrong log strings
Fixes: remove large buffers from stack
ElementText: replace 'char *' with 'std::string'
[win32] AESinkDirectSound: check result of GetStatus()
GUIWindowTestPatternDX::DrawCheckers: check input parameters
AEChannelInfo::BestMatch: fix use boolean in arithmetics
XMLUtils::GetHex: check result of sscanf()
ActiveAEResample::Init: transform to 64bit before shift
CueDocument: replace call of isdigit() with StringUtils::isasciidigit()
cc_decoder: suppress printing to stderr and stdout
[builtins/jsonrpc] Invert default for showing dialogs after 5324.

This is to ensure backward compatibility with skins/add-ons (type="action"),
add-ons directly calling built-in functions, third party applications using the
JSON-RPC API and existing key mappings.
Merge pull request #5045 from Karlson2k/code_fixes_11

Code fixes