[PULL] unreviewed patches
Peter Hutterer
peter.hutterer at who-t.net
Mon May 6 16:44:19 PDT 2013
On Mon, May 06, 2013 at 11:26:10AM -0700, Keith Packard wrote:
> Peter Hutterer <peter.hutterer at who-t.net> writes:
>
> > Xi: fix comment - XI2 grabs aren't keysym grabs
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
>
> > dix: send the current axis value in DeviceChangedEvents (#62321)
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
>
> > Xi: always return BadMatch for XTest devices ChangeDeviceControl requests
>
> (this appears to cover all of the DDX entry points to avoid passing
> XTest devices to them)
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
>
> > If neither HAL nor udev backends are enabled, warn the user
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
>
> > xkb: free XkbRulesUsed and XkbRulesDflt on extension cleanup
>
> I think this should be done in CloseDownDevices -- XkbDeleteRulesDflts
> is already called there, adding a call to your new XkbDeleteRulesUsed
> (which looks fine, btw) seems like the right plan.
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.
>
> > Delete callbacks when extension are closed
> > dix: delete all callbacks before reset
>
> Looks like the second patch makes the first unnecessary?
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.
Cheers,
Peter
> > dix: reset the registry before quitting
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
>
> > os: support pnprintf length modifiers for integers
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
>
> > os: support %% in pnprintf
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
>
> > os: support %c in pnprintf
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
>
> > os: complain about unsupported pnprintf directives
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
>
> > os: Use ErrorFSigSafe from FatalError and it's friends
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
>
> --
> keith.packard at intel.com
More information about the xorg-devel
mailing list