[PULL] unreviewed patches
Keith Packard
keithp at keithp.com
Tue May 7 09:51:02 PDT 2013
Peter Hutterer <peter.hutterer at who-t.net> writes:
> moved there. I assumed your rev-by, I don't want to do another revision. let
> me know if it's not to your liking.
Reviewed-by: Keith Packard <keithp at keithp.com>
> yes, but tbh I do like the idea of extensions cleaning up after themselves
> instead of leaving it to some other piece of code. I can drop the patch
> though, I don't care either way tbh.
Thinking more about it this morning, I do believe it would be better to
not have the extensions duplicate the cleanup work. I think the
duplication makes the API for extensions less clear as I suspect we'll
probably end up with extensions doing it both ways, which is never good
when people are reading through the code.
But, I don't care that deeply about the issue, and the code is correct
in either case -- my comment was more about clarifying the interaction
between the two patches than an actual complaint.
So send whichever version you prefer along, both patches are
Reviewed-by: Keith Packard <keithp at keithp.com>
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20130507/b84e4627/attachment-0001.pgp>
More information about the xorg-devel
mailing list