What are all those "/P"s in the commit message about ?
I think some word is missing in this comment (probably it is 'disabled'):
264 + /* The use of the timeout is currently because some functionality
265 + * is broken which would cause these barrier conditions to never
266 + * eventuate. For now just get to the fail condition */
Maybe ForbidDecorationsOnWindow (Window window); would be better ? You decide.
This comment could be a one-liner (ultraminor):
406 + /* Select for StructureNotify on the parent and wrapper
407 + * windows */
823 - unsigned int width, height, border, depth;
824 + unsigned int width, height, border;
Don't we need depth anymore ? A quick search in the code shows that it was not
really used, although windows should have a depth for advanced animations, no ?
What are all those "/P"s in the commit message about ?
I think some word is missing in this comment (probably it is 'disabled'):
264 + /* The use of the timeout is currently because some functionality
265 + * is broken which would cause these barrier conditions to never
266 + * eventuate. For now just get to the fail condition */
187 + void DisallowDecorat ionsOnWindow (Window window);
Maybe ForbidDecoratio nsOnWindow (Window window); would be better ? You decide.
This comment could be a one-liner (ultraminor):
406 + /* Select for StructureNotify on the parent and wrapper
407 + * windows */
823 - unsigned int width, height, border, depth;
824 + unsigned int width, height, border;
Don't we need depth anymore ? A quick search in the code shows that it was not
really used, although windows should have a depth for advanced animations, no ?