[PATCH] Xnest: Remove socket and its lock file on exit

Jeremy Huddleston jeremyhu at apple.com
Thu Oct 20 14:48:50 PDT 2011


On Oct 20, 2011, at 12:08, Julien Cristau wrote:

>>      if (xnestParentWindow != (Window) 0 &&
>> 	  X.xdestroywindow.window == xnestParentWindow)
>> +	CloseWellKnownConnections();
>> +	OsCleanup(1);
>> 	exit (0);
> That looks broken, don't you need braces around the if branch?  Also I'm
> not quite sure what's the point of doing manual cleanup when you're
> going to exit() on the next line anyway.  So this hunk looks like it
> should be reverted.

Yeah.  I dunno what I was doing there, but it's obviously wrong... 

While your at it, care to review my other pending Xnest change, so I can send both in a [PULL]?

I'm assuming your r-b for the revert of the above hunk.

--Jeremy



More information about the xorg-devel mailing list