[PULL: xserver] Compiler warning fixes for Dtrace probes, DMX

Alan Coopersmith alan.coopersmith at oracle.com
Tue Feb 10 18:36:24 PST 2015


As noted when these were reviewed on xorg-devel in December, they don't fix
all the warnings in the DMX build, but it reduces the number of warnings
reported when building with gcc 4.8.2 on Solaris from 222 to 58.

The following changes since commit 20079c36cf7d377938ca5478447d8b9045cb7d43:

   xkb: Check strings length against request size (2015-02-10 14:28:28 -0800)

are available in the git repository at:

   ssh://people.freedesktop.org/~alanc/xserver master

for you to fetch changes up to 4a758f59a8a65c313244e06852745c8e180ff247:

   dmx: include header for DMXExtensionInit() in dmx.c (2015-02-10 18:14:45 -0800)

----------------------------------------------------------------
Alan Coopersmith (13):
       Get rid of const warnings in XSERVER_INPUT_EVENT dtrace probe calls
       dmx: Add _X_ATTRIBUTE_PRINTF to dmxConfigLog() & dmxConfigOutput()
       dmx: move format strings inline so gcc can check
       dmx: fix printf format argument warnings
       dmx: remove unused variables
       dmx: attempt to untangle nested loops using same index variable
       dmx: more unshadowing of variables
       dmx: glxproxy prototype cleanup
       dmx: remove redundant declaration of dmxFontPrivateIndex from dmxfont.h
       dmx: remove redundant redeclarations of variables from other headers
       dmx: Mark glxIsExtensionSupported as a const char *
       dmx: constify GCOps & GCFuncs pointers
       dmx: include header for DMXExtensionInit() in dmx.c

  dix/Xserver.d                 |    5 ++++-
  dix/dispatch.c                |    5 +----
  dix/events.c                  |    8 +-------
  dix/getevents.c               |    7 +------
  dix/resource.c                |    5 +----
  hw/dmx/config/dmxparse.h      |    3 ++-
  hw/dmx/config/dmxprint.c      |   20 ++++----------------
  hw/dmx/dmx.c                  |    1 +
  hw/dmx/dmxcb.c                |    3 ---
  hw/dmx/dmxfont.h              |    2 --
  hw/dmx/dmxgc.c                |    4 ++--
  hw/dmx/dmxgc.h                |    4 ++--
  hw/dmx/dmxinit.c              |   42 ++++++++++++++++++++---------------------
  hw/dmx/dmxpict.c              |    1 -
  hw/dmx/dmxprop.c              |    6 +++---
  hw/dmx/dmxscrinit.c           |    1 -
  hw/dmx/dmxwindow.c            |    4 ----
  hw/dmx/glxProxy/compsize.h    |    7 +++++++
  hw/dmx/glxProxy/g_disptab.h   |   21 ++++++++++++++++++++-
  hw/dmx/glxProxy/glxcmds.c     |    7 +++----
  hw/dmx/glxProxy/glxcmds.h     |    6 ++++++
  hw/dmx/glxProxy/glxcmdsswap.c |    2 --
  hw/dmx/glxProxy/glxext.h      |    1 +
  hw/dmx/glxProxy/glxscreens.c  |    6 +++---
  hw/dmx/glxProxy/glxscreens.h  |    2 ++
  hw/dmx/glxProxy/glxserver.h   |    8 --------
  hw/dmx/glxProxy/glxsingle.c   |    6 ++----
  hw/dmx/glxProxy/glxsingle.h   |    4 ----
  hw/dmx/glxProxy/glxswap.c     |    4 +---
  hw/dmx/glxProxy/glxvendor.c   |    6 ++----
  hw/dmx/input/dmxinputinit.c   |   10 +++++-----
  include/Makefile.am           |    1 +
  include/probes.h              |   41 ++++++++++++++++++++++++++++++++++++++++
  os/connection.c               |   12 +++---------
  34 files changed, 140 insertions(+), 125 deletions(-)
  create mode 100644 include/probes.h

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list