[PULL] mi, ChangeGC, clientErrorValue, noClientException cleanups

Jamey Sharp jamey at minilop.net
Thu May 13 17:26:00 PDT 2010


Keith reviewed the rest of this series, so it's ready for merge now.
Thanks to keithp, ajax, daniels, and Eamon for reviewing this series
along the way!

The following changes since commit c9e7ca4404803fe44d4684e0bb2ca2ee10fd4bb3:
  Eamon Walsh (1):
        xselinux: Remove use of devPrivates init/free callbacks.

are available in the git repository at:

  git://people.freedesktop.org/~jamey/xserver cleanups

Jamey Sharp (11):
      miwideline: Factor out span buffer allocation.
      Simplify miFillPolyHelper and miLineArc.
      Don't statically allocate the ChangeGC parameter array.
      dixChangeGC callers: Use ChangeGCVal instead of XID almost everywhere.
      Pre-validate ChangeGC XIDs.
      Replace dixChangeGC with calls directly to the right variant.
      Define GCAllBits as the union of all valid CreateGC masks.
      Quit using clientErrorValue in dix/gc.c.
      clientErrorValue is never used outside dix. Stop importing it.
      Quit using clientErrorValue in dix/colormap.c.
      Eliminate boilerplate around client->noClientException.

 Xext/bigreq.c                         |    2 +-
 Xext/dpms.c                           |   16 +-
 Xext/geext.c                          |    2 +-
 Xext/panoramiX.c                      |   12 +-
 Xext/panoramiXprocs.c                 |   37 ++--
 Xext/saver.c                          |    6 +-
 Xext/security.c                       |    7 +-
 Xext/shape.c                          |    8 +-
 Xext/shm.c                            |   18 +-
 Xext/sync.c                           |   14 +-
 Xext/xcalibrate.c                     |    6 +-
 Xext/xcmisc.c                         |    6 +-
 Xext/xf86bigfont.c                    |    4 +-
 Xext/xres.c                           |    8 +-
 Xext/xselinux_ext.c                   |    6 +-
 Xext/xtest.c                          |    8 +-
 Xext/xvdisp.c                         |    4 +-
 Xext/xvmain.c                         |    2 -
 Xi/exevents.c                         |    2 +-
 composite/compalloc.c                 |    5 +-
 composite/compext.c                   |   10 +-
 damageext/damageext.c                 |   10 +-
 dbe/dbe.c                             |    8 +-
 dix/colormap.c                        |   26 +--
 dix/devices.c                         |    8 +-
 dix/dispatch.c                        |  377 +++++++++++----------------------
 dix/dixfonts.c                        |   14 +-
 dix/extension.c                       |    4 +-
 dix/gc.c                              |  253 ++++++++++------------
 dix/glyphcurs.c                       |    5 +-
 dix/property.c                        |   14 +-
 dix/selection.c                       |    8 +-
 dix/window.c                          |   21 +-
 fb/fbseg.c                            |    4 +-
 glx/glxdriswrast.c                    |   16 +-
 glx/glxext.c                          |    2 +-
 glx/xfont.c                           |    2 -
 hw/dmx/dmx.c                          |   28 ++--
 hw/dmx/dmxfont.c                      |    2 -
 hw/kdrive/ephyr/ephyr_draw.c          |   18 +-
 hw/kdrive/ephyr/ephyrdriext.c         |   24 +-
 hw/kdrive/src/kcmap.c                 |    2 +-
 hw/kdrive/src/kxv.c                   |    8 +-
 hw/vfb/InitOutput.c                   |    2 +-
 hw/xfree86/common/xf86xv.c            |   15 +-
 hw/xfree86/dixmods/extmod/xf86dga2.c  |   64 +++---
 hw/xfree86/dixmods/extmod/xf86vmode.c |   44 ++--
 hw/xfree86/dri/xf86dri.c              |   24 +-
 hw/xfree86/dri2/dri2ext.c             |   36 ++--
 hw/xfree86/vgahw/vgaCmap.c            |    2 +-
 hw/xfree86/xaa/xaaPCache.c            |    8 +-
 hw/xquartz/applewm.c                  |   26 ++--
 hw/xquartz/pseudoramiX.c              |   10 +-
 hw/xquartz/xpr/appledri.c             |   14 +-
 hw/xwin/winwindowswm.c                |   14 +-
 include/colormap.h                    |    6 +-
 include/gc.h                          |    9 +-
 include/gcstruct.h                    |    2 +
 mi/miarc.c                            |   36 ++--
 mi/mibitblt.c                         |   41 ++--
 mi/midispcur.c                        |   24 +-
 mi/miexpose.c                         |   11 +-
 mi/miglblt.c                          |   32 ++--
 mi/mipolypnt.c                        |   12 +-
 mi/miwideline.c                       |  203 ++++++------------
 mi/miwideline.h                       |    8 +-
 mi/mizerarc.c                         |    8 +-
 miext/cw/cw.c                         |   22 +-
 randr/rrcrtc.c                        |   14 +-
 randr/rrdispatch.c                    |    2 +-
 randr/rrmode.c                        |    2 +-
 randr/rroutput.c                      |    6 +-
 randr/rrproperty.c                    |   14 +-
 randr/rrscreen.c                      |    8 +-
 randr/rrxinerama.c                    |   12 +-
 record/record.c                       |    4 +-
 render/miindex.c                      |    2 +-
 render/mirect.c                       |   23 +-
 render/render.c                       |   50 ++---
 xfixes/cursor.c                       |   16 +-
 xfixes/region.c                       |   57 ++---
 xfixes/saveset.c                      |    6 +-
 xfixes/xfixes.c                       |    2 +-
 xkb/xkb.c                             |   50 +++---
 84 files changed, 863 insertions(+), 1115 deletions(-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100513/ba798a62/attachment.pgp>


More information about the xorg-devel mailing list