Created attachment 622768 [details] [diff] [review]
This may be a 10.8/clang only issue, but when I tried to build yesterday, I got an error when casting nil to NSOpenGLPixelFormatAttribute (aka unsigned int). That's being used to terminate the array of attributes.
According to the docs, -[NSOpenGLPixelFormat initWithAttributes] expects a 0 terminated array which we aren't explicitly doing in a couple cases - instead we're casting nil.
We do end up doing this correctly later in GLContextProviderCGL.mm