aboutsummaryrefslogtreecommitdiff
path: root/debian
diff options
context:
space:
mode:
authorKeith Packard <keithp@keithp.com>2015-07-04 01:22:02 +0200
committerMihai Moldovan <ionic@ionic.de>2015-07-04 02:20:17 +0200
commit558e45c1516f66d8084215b319d214f94faf09bc (patch)
tree6bc1a9aa735c060f3d95e50c46a517a2a85ff2ea /debian
parentab58e5781d4e64169b8f6931bf7535cbcf439573 (diff)
downloadnx-libs-558e45c1516f66d8084215b319d214f94faf09bc.tar.gz
nx-libs-558e45c1516f66d8084215b319d214f94faf09bc.tar.bz2
nx-libs-558e45c1516f66d8084215b319d214f94faf09bc.zip
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 <keithp@keithp.com> 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)
Diffstat (limited to 'debian')
-rw-r--r--debian/changelog7
-rw-r--r--debian/patches/1258_nx-X11_randr-Clean-up-compiler-warnings-about-u.full.patch117
-rw-r--r--debian/patches/series1
3 files changed, 125 insertions, 0 deletions
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 <git-admin@x2go.org> 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 <keithp@keithp.com>
+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 <keithp@keithp.com>
+
+ 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