GHOST_Context.cpp seems fine, not sure why other changes are made, (initialized to dummy values, never used)

Changes in GHOST_Context.cpp have already been committed in the mean time.
https://developer.blender.org/rBdd897de06114d6a39908f82c228027f633359d0b

Revisions in GHOST_ContextWGL.cpp are intended to fix compilation errors reported by @Joshua Leung (aligorith):
http://lists.blender.org/pipermail/bf-committers/2014-October/044322.html

intern\ghost\intern\GHOST_ContextWGL.cpp: In member function 'void GHOST_ContextWGL::initContextWGLEW(PIXELFORMATDESCRIPTOR&)':
intern\ghost\intern\GHOST_ContextWGL.cpp:400:1: warning: jump to label 'finalize' [-fpermissive]
intern\ghost\intern\GHOST_ContextWGL.cpp:356:8: warning: from here [-fpermissive]
intern\ghost\intern\GHOST_ContextWGL.cpp:358:6: error: crosses initialization of 'int iPixelFormat'

I see, the values aren't used in this case, (the goto isnt reading the values)... its a bit messy but since its is simple working code, just leave it?...

That, or get a full mingw build log from ghost/win32 and collect warnings and quiet (all/most) of them.
Rather not do this piecemeal.

Just confirming that we had the same issues as Aligorith and kjym3 here with Windows/MinGW64 and our own patch was nessecary to fix this.

I think I've fixed these issues now.

I decided to remove all intermixed decls in functions that use goto, for consistency.

I would have used something like boost::optional to manage these resources, but the protocol isn't that simple.

I think I could commit all of these remaining changes. My only question is whether the change to the type of embedder window should also be changed for msvc when compiling for 64 bit.