[Xcb] [PATCH] XCBify xlsatoms

Peter Harris git at peter.is-a-geek.org
Wed Nov 11 19:33:47 PST 2009


On Wed, Nov 11, 2009 at 9:04 PM, Jeremy Huddleston wrote:
> I'd rather see these xcb versions have a configure option (--enable-xcb for example)... I'm fine with it being on by default, but it's nice to have a fallback and easy regression testing.

It seems a lot of work for minimal gain. Personally, I would find it
easier to check out the 1.x branch (and/or download the 1.x tarball)
than to pass a magic option to configure. Especially given:

$ wc -l xlsclients.c 0001-xlsclients.patch
     320 xlsclients.c
     690 0001-xlsclients.patch

$ wc -l xlsatoms.c 0001-Convert-xlsatoms-to-XCB.patch
     234 xlsatoms.c
     254 0001-Convert-xlsatoms-to-XCB.patch

These are effectively rewrites. To leave the code in a readable state,
you'd wind up with the Xlib version inside one giant #ifdef, and the
XCB version inside the #else (or vice versa).

If these were larger programs, I'd agree with you. But these are very
small wrappers around basic X protocol operations. There's really very
little (mostly just argument parsing and the copyright notice) that
wasn't Xlib specific inside either of them.

The last time this came up:
 http://lists.freedesktop.org/archives/xcb/2009-September/005014.html
 http://lists.freedesktop.org/archives/xcb/2009-September/005019.html

Peter Harris


More information about the xorg-devel mailing list