[Xquartz-dev] 10.8 issue with non-standard code

[Jeremy Huddleston Sequoia]

+QuartzModeInfoPtr *lastUsedMode;
...
+ QuartzRandRSetMode(pScreen, &lastUsedMode, TRUE);

That won't work. You should see a warning from the compiler telling you
that you're passing incompatible pointer types. Seeing that warning 
should
be a huge red flag that something is wrong.

You probably want this for lastUsedMode:

QuartzModeInfo lastUsedMode;

---

It also looks like you may be using lastUsedMode uninitialized.

It was probably the case that CG just returned an error in Lion and is
crashing now in ML.  That crash is a bug in CG, but it's exposed by a 
bug
in your code.

--Jeremy

On 2012-08-27 18:46, doh123 wrote:

> I *think* this is all i did to get it working... but I haven't tested
> this. I believe its all the changes I did to get the toggle working 
> on
> 10.5 - 10.7 fine. I keep my stuff in patch files anyways, but all of 
> this
> wasn't in the same one so I had to go through and find all the right
> parts, but If I got it right, it adds CMD+Opt+A toggle back into
> fullscreen, just something changed with 10.8, and I think its 
> something
> actually with the actual capturing the display, but I may be wrong... 
> or
> it may add some other issues I've never run in to... because I'm not 
> sure
> why this was all set to disabled in the first place.