Catch broken configurations inside of the Post Processing shaders.

This catches most instances of configuration failures that can happen in a post processing shader.
Gives a user a helpful error message that lets them know what they have failed to set up correctly
DolphinWX: Move the CodeView debugger view over to wxGraphicsContext

This is a more advanced drawing 'backend' over the previous one and allows us to control things like transparency and anti-aliasing, etc.
DolphinWX: Parenthesize some expressions in CodeView.cpp
Merge pull request #866 from comex/warnings2

Remove unused variable m_zero.
Merge pull request #828 from Sonicadvance1/pp-shader-catch-broken-config

Catch broken configurations inside of the Post Processing shaders.
Revert "Catch broken configurations inside of the Post Processing shaders."
Merge pull request #872 from dolphin-emu/revert-828-pp-shader-catch-broken-config

Revert "Catch broken configurations inside of the Post Processing shader...
JIT: fix incredibly silly mistake in fmul rounding patch
Merge pull request #865 from lioncash/debugger-stuff

DolphinWX: Use wxGraphicsContext in the Code View for the debugger.
Merge pull request #874 from FioraAeterna/fixidiocy

JIT: fix incredibly silly mistake in fmul rounding patch
CommandProcessor: mark some functions as static
Merge pull request #838 from Tilka/unused

CommandProcessor: mark some functions as static
DiscIO: Make the unordered set in IsSoundFile static

Doesn't need to be instantiated every time the function is called.
Merge pull request #875 from lioncash/statics

DiscIO: Make the unordered set in IsSoundFile static
PPCFP: add comment
Merge pull request #884 from FioraAeterna/ppcfpopt

PPCFP: add comment
Refactor SetCpStatus into two functions for from-GPU and from-CPU mode rather than a boolean parameter.

This shouldn't affect functionality.  I'm not sure if the breakpoint
distinction is actually necessary (my commit messages from the old
dc-netplay last year claim that breakpoints are broken anyway, but I
don't remember why), but I don't actually need to change this part of
the code (yet), so I'll stick with the trimmings change for now.
A tiny restructuring to allow inlining of FifoCommandRunnable.  Probably useless.
Add Flag support to ChunkFile.h
Unify three types of non-FIFO requests to the GPU thread around Common::Event and Common::Flag.

The only possible functionality change is that s_efbAccessRequested and
s_swapRequested are no longer reset at init and shutdown of the OGL
backend (only; this is the only interaction any files other than
MainBase.cpp have with them).  I am fairly certain this was entirely
vestigial.

Possible performance implications: efbAccessReady now uses an Event
rather than spinning, which might be slightly slower, but considering
the slow loop the flags are being checked in from the GPU thread, I
doubt it's noticeable.

Also, this uses sequentially consistent rather than release/acquire
memory order, which might be slightly slower, especially on ARM...
something to improve in Event/Flag, really.