From 558e45c1516f66d8084215b319d214f94faf09bc Mon Sep 17 00:00:00 2001 From: Keith Packard Date: Sat, 4 Jul 2015 01:22:02 +0200 Subject: nx-X11: Backport: randr: Clean up compiler warnings about unused and shadowing variables set but not used variables shadowing a previous local A hidden problem was that the VERIFY_RR_* macros define local 'rc' variables, any other local definitions for those would be shadowed and generate warnings from gcc. I've renamed the other locals 'ret' instead of 'rc'. Signed-off-by: Keith Packard Backported from Arctica GH 3.6.x branch. v2: backport to nx-libs 3.6.x (Ulrich Sibiller) v3: backport to nx-libs 3.5.0.x (Mihai Moldovan) --- debian/changelog | 7 ++ ...r-Clean-up-compiler-warnings-about-u.full.patch | 117 +++++++++++++++++++++ debian/patches/series | 1 + 3 files changed, 125 insertions(+) create mode 100644 debian/patches/1258_nx-X11_randr-Clean-up-compiler-warnings-about-u.full.patch diff --git a/debian/changelog b/debian/changelog index a0babafc0..ddfecbd87 100644 --- a/debian/changelog +++ b/debian/changelog @@ -311,6 +311,13 @@ nx-libs (2:3.5.0.32-0x2go1) UNRELEASED; urgency=low v3: backport to nx-libs 3.5.0.x (Mihai Moldovan) Adds: - 1257_nx-X11_randr-Fix-REQUEST-vs-REQUEST_SIZE_MATCH-.full.patch + * nx-X11: randr: Clean up compiler warnings about unused and shadowing + variables + Backported from Arctica GH 3.6.x branch. + v2: backport to nx-libs 3.6.x (Ulrich Sibiller) + v3: backport to nx-libs 3.5.0.x (Mihai Moldovan) + Adds: + - 1258_nx-X11_randr-Clean-up-compiler-warnings-about-u.full.patch -- X2Go Release Manager Tue, 17 Mar 2015 19:19:32 +0100 diff --git a/debian/patches/1258_nx-X11_randr-Clean-up-compiler-warnings-about-u.full.patch b/debian/patches/1258_nx-X11_randr-Clean-up-compiler-warnings-about-u.full.patch new file mode 100644 index 000000000..38869ca9d --- /dev/null +++ b/debian/patches/1258_nx-X11_randr-Clean-up-compiler-warnings-about-u.full.patch @@ -0,0 +1,117 @@ +commit 9dbd74793f4b125d56e3d0f63a845b5f7550d3b4 +Author: Keith Packard +Date: Thu Jun 21 18:42:46 2012 -0700 + + Backport: randr: Clean up compiler warnings about unused and shadowing variables + + set but not used variables + shadowing a previous local + + A hidden problem was that the VERIFY_RR_* macros define local 'rc' + variables, any other local definitions for those would be shadowed and + generate warnings from gcc. I've renamed the other locals 'ret' + instead of 'rc'. + + Signed-off-by: Keith Packard + + Backported from Arctica GH 3.6.x branch. + + v2: backport to nx-libs 3.6.x (Ulrich Sibiller) + v3: backport to nx-libs 3.5.0.x (Mihai Moldovan) + +diff --git a/nx-X11/programs/Xserver/randr/rrcrtc.c b/nx-X11/programs/Xserver/randr/rrcrtc.c +index fb82a80..8a34962 100644 +--- a/nx-X11/programs/Xserver/randr/rrcrtc.c ++++ b/nx-X11/programs/Xserver/randr/rrcrtc.c +@@ -642,7 +642,6 @@ ProcRRSetCrtcConfig (ClientPtr client) + int numOutputs; + RROutputPtr *outputs = NULL; + RROutput *outputIds; +- TimeStamp configTime; + TimeStamp time; + Rotation rotation; + int i, j; +@@ -745,7 +744,6 @@ ProcRRSetCrtcConfig (ClientPtr client) + pScrPriv = rrGetScrPriv(pScreen); + + time = ClientTimeToServerTime(stuff->timestamp); +- configTime = ClientTimeToServerTime(stuff->configTimestamp); + + if (!pScrPriv) + { +@@ -754,19 +752,6 @@ ProcRRSetCrtcConfig (ClientPtr client) + goto sendReply; + } + +-#if 0 +- /* +- * if the client's config timestamp is not the same as the last config +- * timestamp, then the config information isn't up-to-date and +- * can't even be validated +- */ +- if (CompareTimeStamps (configTime, pScrPriv->lastConfigTime) != 0) +- { +- rep.status = RRSetConfigInvalidConfigTime; +- goto sendReply; +- } +-#endif +- + /* + * Validate requested rotation + */ +diff --git a/nx-X11/programs/Xserver/randr/rrinfo.c b/nx-X11/programs/Xserver/randr/rrinfo.c +index 7e77d39..470e1bb 100644 +--- a/nx-X11/programs/Xserver/randr/rrinfo.c ++++ b/nx-X11/programs/Xserver/randr/rrinfo.c +@@ -83,6 +83,7 @@ RRScanOldConfig (ScreenPtr pScreen, Rotation rotations) + int i; + CARD16 minWidth = MAXSHORT, minHeight = MAXSHORT; + CARD16 maxWidth = 0, maxHeight = 0; ++ CARD16 width, height; + + /* + * First time through, create a crtc and output and hook +@@ -153,11 +154,11 @@ RRScanOldConfig (ScreenPtr pScreen, Rotation rotations) + /* find size bounds */ + for (i = 0; i < output->numModes + output->numUserModes; i++) + { +- RRModePtr mode = (i < output->numModes ? ++ mode = (i < output->numModes ? + output->modes[i] : + output->userModes[i-output->numModes]); +- CARD16 width = mode->mode.width; +- CARD16 height = mode->mode.height; ++ width = mode->mode.width; ++ height = mode->mode.height; + + if (width < minWidth) minWidth = width; + if (width > maxWidth) maxWidth = width; +diff --git a/nx-X11/programs/Xserver/randr/rrmode.c b/nx-X11/programs/Xserver/randr/rrmode.c +index 43ffe07..db0af33 100644 +--- a/nx-X11/programs/Xserver/randr/rrmode.c ++++ b/nx-X11/programs/Xserver/randr/rrmode.c +@@ -194,7 +194,7 @@ RRModesForScreen (ScreenPtr pScreen, int *num_ret) + for (o = 0; o < pScrPriv->numOutputs; o++) + { + RROutputPtr output = pScrPriv->outputs[o]; +- int m, n; ++ int n; + + for (m = 0; m < output->numModes + output->numUserModes; m++) + { +@@ -299,7 +299,6 @@ ProcRRCreateMode (ClientPtr client) + xRRCreateModeReply rep; + WindowPtr pWin; + ScreenPtr pScreen; +- rrScrPrivPtr pScrPriv; + xRRModeInfo *modeInfo; + long units_after; + char *name; +@@ -317,7 +316,6 @@ ProcRRCreateMode (ClientPtr client) + return rc; + + pScreen = pWin->drawable.pScreen; +- pScrPriv = rrGetScrPriv(pScreen); + + modeInfo = &stuff->modeInfo; + name = (char *) (stuff + 1); diff --git a/debian/patches/series b/debian/patches/series index 38fa1072b..150a3033a 100644 --- a/debian/patches/series +++ b/debian/patches/series @@ -144,6 +144,7 @@ 1255_nx-X11_RRModeCreate-plug-memory-leak-of-newMode.full.patch 1256_nx-X11_ProcRRGetScreenInfo-swap-configTimestamp.full.patch 1257_nx-X11_randr-Fix-REQUEST-vs-REQUEST_SIZE_MATCH-.full.patch +1258_nx-X11_randr-Clean-up-compiler-warnings-about-u.full.patch 9900-dxpc-license-history.full+lite.patch 0016_nx-X11_install-location.debian.patch 0102_xserver-xext_set-securitypolicy-path.debian.patch -- cgit v1.2.3