On Mon, 30 Nov 2009 14:37:28 +1000, Peter Hutterer <[email protected]> 
wrote:

> Gaetan Nadon (1):
>       configure.ac: error while checking for XDMXCONFIG_DEP

No S-o-b line from Gaetan? I found a few other commits missing S-o-b
lines from the author recently; looks like I need to be more careful.

> Peter Hutterer (3):
>       dix: fix memory leak, free event list on shutdown. (#25028)

I'd rather see this as:

    FreeEventList(InputEventList, InputEventListLen);
    InputEventList = NULL;
    InputEventListLen = 0;

As you can see, the bug was caused by an open-coded copy of FreeEventList.

>       dix: remove some obsolete comment.

Reviewed-by: Keith Packard <[email protected]>

>       dix: remove core devices when shutting down. (#25028)

Reviewed-by: Keith Packard <[email protected]>

(A bit ugly, but meh)

>  configure.ac  |   14 +++++++++-----
>  dix/devices.c |    4 +++-
>  dix/events.c  |    2 ++
>  3 files changed, 14 insertions(+), 6 deletions(-)

-- 
[email protected]

Attachment: pgpxZD5vvDAnz.pgp
Description: PGP signature

_______________________________________________
xorg-devel mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to