On Thu, March 27, 2008 12:13 pm, Jason Tackaberry wrote:
> On Thu, 2008-03-27 at 08:21 +0000, Adam Charrett wrote:
>> This patch adds several new methods to X11Window which adds the ability
>> to set the shape mask for the window (either from a bitmask string or an
>> imlib2 image) and disable/enable windows decorations. I've also included
>> a test app that shows the new features in use.
>
> Thanks for the patch.  It looks mostly good (although the formatting
> style is slightly different than the existing code, this can be
> changed), except for this:

Ahh, love those different formatting styles, fell into the bad habit of
using my preferred style rather than maintaining the style in the file

>
>> -    { "set_title", (PyCFunction)X11Window_PyObject__set_title,
>> METH_VARARGS },
>> -    { "get_title", (PyCFunction)X11Window_PyObject__get_title,
>> METH_VARARGS },
>> -    { "get_children", (PyCFunction)X11Window_PyObject__get_children,
>> METH_VARARGS },
>> -    { "get_parent", (PyCFunction)X11Window_PyObject__get_parent,
>> METH_VARARGS },
>> -    { "get_properties",
>> (PyCFunction)X11Window_PyObject__get_properties, METH_VARARGS },
>> +    { "set_shape_mask",
>> (PyCFunction)X11Window_PyObject__set_shape_mask, METH_VARARGS },
>> +    { "reset_shape_mask",
>> (PyCFunction)X11Window_PyObject__reset_shape_mask, METH_VARARGS },
>> +    { "set_decorated", (PyCFunction)X11Window_PyObject__set_decorated,
>> METH_VARARGS },
>
> Why are you removing all these methods?
>

Not intentional, bad patch, seems that when I merge changes I had done
while at work with the svn version I accidentally remove some methods,
doh.
Can't recreate the patch now but will try when I get home.

Cheers

Adam


-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel

Reply via email to