aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* mi: fix shadow warningsUlrich Sibiller2020-01-102-9/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Based on the following commit. But for mispans.c I think the first contained fix is wrong (nested loop with variable i) so I took another approach. commit f02e27e4fcc34413b2051e5a01edc92172fa8612 Author: Yaakov Selkowitz <yselkowitz@users.sourceforge.net> Date: Tue Oct 16 02:16:17 2012 -0500 mi: fix shadow warnings mibitblt.c: In function 'miGetImage': mibitblt.c:617:20: warning: declaration of 'pt' shadows a previous local mibitblt.c:609:17: warning: shadowed declaration is here mispans.c: In function 'miFillUniqueSpanGroup': mispans.c:456:33: warning: declaration of 'i' shadows a previous local mispans.c:382:9: warning: shadowed declaration is here mispans.c:488:17: warning: declaration of 'i' shadows a previous local mispans.c:382:9: warning: shadowed declaration is here Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net> Fixes these warnings: mi/mispans.c: fix shadowed vars mispans.c: In function ‘miFillUniqueSpanGroup’: mispans.c:465:12: warning: declaration of ‘i’ shadows a previous local [-Wshadow=compatible-local] int i; ^ mispans.c:387:21: note: shadowed declaration is here register int i; ^ mispans.c:497:10: warning: declaration of ‘i’ shadows a previous local [-Wshadow=compatible-local] int i; ^ mispans.c:387:21: note: shadowed declaration is here register int i; ^ mibitblt.c: In function ‘miGetImage’: mibitblt.c:669:13: warning: declaration of ‘pt’ shadows a previous local [-Wshadow=compatible-local] xPoint pt; ^~ mibitblt.c:659:18: note: shadowed declaration is here DDXPointRec pt = {0, 0}; ^~
* _XkbFilterDeviceBtn: move variable declarations to match usage scopeUlrich Sibiller2020-01-101-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix this warning: xkbActions.c: In function ‘_XkbFilterDeviceBtn’: xkbActions.c:1084:6: warning: declaration of ‘button’ shadows a previous local [-Wshadow=compatible-local] int button; ^~~~~~ xkbActions.c:1044:6: note: shadowed declaration is here int button; ^~~~~~ Backport of this xorg-xserver upstream commit: commit 41dd7cf673e47ab74c5065b8dbf268b2e5facb64 Author: Alan Coopersmith <alan.coopersmith@oracle.com> Date: Mon Dec 12 16:49:34 2011 -0800 _XkbFilterDeviceBtn: move variable declarations to match usage scope The main body of this function is an if { } else if { } pair of blocks. Previously there was int button at the top level scope which is used only in the first block, and a redeclaration of int button inside the second block. Since there's no overlap in the code paths for the two uses of button, move the one from the outer block into the first block to help the programmer more quickly determine they are unrelated usages, and to silence the gcc warning of: xkbActions.c: In function '_XkbFilterDeviceBtn': xkbActions.c:999:6: warning: declaration of 'button' shadows a previous local xkbActions.c:955:6: warning: shadowed declaration is here For consistency, move DeviceIntPtr dev declarations as well that are used in the same way. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jamey Sharp <jamey@minilop.net> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
* XkbFindSrvLedInfo: remove extraneous name-clashing sli variableUlrich Sibiller2020-01-101-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix shadow warning: xkbLEDs.c: In function ‘XkbFindSrvLedInfo’: xkbLEDs.c:633:19: warning: declaration of ‘sli’ shadows a previous local [-Wshadow=compatible-local] XkbSrvLedInfoPtr sli; ^~~ xkbLEDs.c:629:18: note: shadowed declaration is here XkbSrvLedInfoPtr sli; ^~~ Backport of this xorg-xserver commit: commit d8eb9b5faa2d2c9350bdaffef8749a9cac3c87b8 Author: Alan Coopersmith <alan.coopersmith@oracle.com> Date: Mon Dec 12 16:49:34 2011 -0800 XkbFindSrvLedInfo: remove extraneous name-clashing sli variable Variable is already defined outside the outer if statement, and there's no need to redefine inside the if statement. No point in setting sli before if (dev->kbdfeed->xkb_sli==NULL) check - if check is true, we immediately set it, if check is false, we immediately return without further reference or use of it. The one thing we do with it inside the inner if statement is store an allocation in it for a brief moment before writing to the final destination, which is immediately returned to the caller. In short, there's no benefit to the variable at all in this block, it just gives the optimizer more code to figure out how to omit. Fixes gcc warning: xkbLEDs.c: In function 'XkbFindSrvLedInfo': xkbLEDs.c:683:19: warning: declaration of 'sli' shadows a previous local xkbLEDs.c:679:18: warning: shadowed declaration is here Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jamey Sharp <jamey@minilop.net> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
* xkb: fix shadow warningsUlrich Sibiller2020-01-105-16/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | xkb.c: In function ‘ProcXkbSetCompatMap’: xkb.c:2601:20: warning: declaration of ‘i’ shadows a previous local [-Wshadow=compatible-local] register unsigned i,bit; ^ xkb.c:2529:23: note: shadowed declaration is here register unsigned i,bit; ^ xkb.c:2601:22: warning: declaration of ‘bit’ shadows a previous local [-Wshadow=compatible-local] register unsigned i,bit; ^~~ xkb.c:2529:25: note: shadowed declaration is here register unsigned i,bit; ^~~ Backport of this xorg upstream commit: commit 344eea237fc07dedfd733d14f95ed0ad26bb5f81 Author: Yaakov Selkowitz <yselkowitz@users.sourceforge.net> Date: Tue Oct 16 02:04:34 2012 -0500 xkb: fix shadow warnings Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
* CheckForEmptyMask does not need to declare int n twiceUlrich Sibiller2020-01-101-1/+1
| | | | | | | | | | | | | | | | | | | | commit 424dbde891486ad6a6c00c61a334031ff18f5556 Author: Alan Coopersmith <alan.coopersmith@oracle.com> Date: Fri Dec 9 10:08:55 2011 -0800 CheckForEmptyMask does not need to declare int n twice Just use the existing n variable again in the ARGB_CURSOR loop instead of creating another one. Fixes gcc -Wshadow warning: cursor.c: In function 'CheckForEmptyMask': cursor.c:155:6: warning: declaration of 'n' shadows a previous local cursor.c:146:9: warning: shadowed declaration is here Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jamey Sharp <jamey@minilop.net> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
* Drawable.c: fix logic broken by scope cleanupUlrich Sibiller2020-01-101-5/+3
| | | | | | | | | | | | xp must be declared outside the while loop! xp was reset in each interation despite being increased at the iteration's end! To protect against this happening again replace the while loop by a for loop that references xp. Bug introduced by d94f472744355c71987d145727aecace1e8455b6
* Cursor.c: drop unused return code for nxagentDisconnectAllCursorUlrich Sibiller2020-01-093-4/+4
|
* Pixmap.c: drop unused return code for nxagentDisconnectAllPixmapsUlrich Sibiller2020-01-092-12/+7
|
* Pixmap.c: fix variable shadowingUlrich Sibiller2020-01-092-2/+16
| | | | | | | | | | | | | | | | | Pixmap.c: In function ‘nxagentDisconnectAllPixmaps’: Pixmap.c:677:19: warning: declaration of ‘r’ shadows a previous local [-Wshadow=compatible-local] for (int i = 0, r = 1; i < MAXCLIENTS; r = 1, i++) ^ Pixmap.c:652:7: note: shadowed declaration is here int r = 1; ^ Pixmap.c: In function ‘nxagentReconnectAllPixmaps’: Pixmap.c:840:19: warning: declaration of ‘result’ shadows a previous local [-Wshadow=compatible-local] for (int i = 0, result = 1; i < MAXCLIENTS; result = 1, i++) ^~~~~~ Pixmap.c:807:8: note: shadowed declaration is here Bool result = 1; ^~~~~~
* Display.c: fix shadowed/unused variableUlrich Sibiller2020-01-091-1/+0
| | | | | | | | | | | Display.c: In function ‘nxagentInitAndCheckVisuals’: Display.c:2273:10: warning: declaration of ‘matched’ shadows a previous local [-Wshadow=compatible-local] bool matched = false; ^~~~~~~ Display.c:2258:8: note: shadowed declaration is here bool matched; ^~~~~~~ Display.c:2258:8: warning: unused variable ‘matched’ [-Wunused-variable]
* Events.c: fix two shadowed variablesUlrich Sibiller2020-01-091-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Events.c: In function ‘nxagentDispatchEvents’: Events.c:1241:21: warning: declaration of ‘pScreen’ shadows a previous local [-Wshadow=compatible-local] ScreenPtr pScreen = nxagentScreen(X.xbutton.window); ^~~~~~~ Events.c:827:13: note: shadowed declaration is here ScreenPtr pScreen = NULL; ^~~~~~~ Events.c:1383:19: warning: declaration of ‘pScreen’ shadows a previous local [-Wshadow=compatible-local] ScreenPtr pScreen = nxagentScreen(X.xmotion.window); ^~~~~~~ Events.c:827:13: note: shadowed declaration is here ScreenPtr pScreen = NULL; ^~~~~~~ Events.c: In function ‘nxagentDispatchEvents’: Events.c:1383:19: warning: declaration of ‘pScreen’ shadows a previous local [-Wshadow=compatible-local] ScreenPtr pScreen = nxagentScreen(X.xmotion.window); ^~~~~~~ Events.c:827:13: note: shadowed declaration is here ScreenPtr pScreen = NULL; ^~~~~~~
* Clipboard.c: fix shadowed variableUlrich Sibiller2020-01-091-6/+8
| | | | | | | | | | | | this was harmless but not nice nevertheless. Clipboard.c: In function ‘nxagentConvertSelection’: Clipboard.c:1850:9: warning: declaration of ‘i’ shadows a previous local [-Wshadow=compatible-local] int i = nxagentFindCurrentSelectionIndex(selection); ^ Clipboard.c:1736:7: note: shadowed declaration is here int i = nxagentFindCurrentSelectionIndex(selection); ^
* nxagent: build with -Wshadow to detect shadowing variablesUlrich Sibiller2020-01-093-4/+4
|
* Drawable.c: fix segfaultUlrich Sibiller2020-01-091-1/+1
| | | | | | | Revert a wrong change that misteriously slipped in via the comment reformatting commit 0133d4619. Fixes ArcticaProject/nx-libs#893
* Rootless.c: fix wring datatype being usedUlrich Sibiller2020-01-081-2/+2
| | | | | | | | | Using XlibAtom here is wrong. On a 64bit system the size of this type is 8 bytes (64bit). At this stage we do not read Xlib datatypes but wire datatypes. Over the wire an Atom is always 32bit, so we need to use CARD32 here. Fixes ArcticaProject/nx-libs#894
* hw/nxagent/Splash.c Fix build on 32bit architecture.Mike Gabriel2020-01-061-10/+3
| | | | | | | Regression fix for 427b7b9777727df4115e3e9e6e63d5176fb0b495. As suggested by Ulrich Sibiller. Fixes ArcticaProject/nx-libs#891.
* Merge branch 'uli42-pr/various3' into 3.6.xMike Gabriel2020-01-0646-1187/+526
|\ | | | | | | Attributes GH PR #880: https://github.com/ArcticaProject/nx-libs/pull/880
| * Atoms.c: describe atoms usageUlrich Sibiller2020-01-061-0/+2
| |
| * Atoms.c: describe atoms usageUlrich Sibiller2020-01-061-11/+44
| |
| * Display.c: Fix broken code in TEST caseUlrich Sibiller2020-01-061-2/+1
| |
| * Image.c: undo wrong string replacement nxagentBitmapBitOrderUlrich Sibiller2020-01-061-1/+1
| | | | | | | | | | | | nxagentBitmapBitOrder never existed and should really be BitmapBitOrder. It was accidently replaced in 523009b96bd2e479c9823267bc060f8e5b0516e8
| * Display.c: add a describing comment for nxagentDisplayErrorPredicateUlrich Sibiller2020-01-061-0/+14
| |
| * Screen.c: harmonize size/wmhints codeUlrich Sibiller2020-01-061-21/+11
| |
| * Events.c: pass a Boolean to nxagentSwitchFullscreenUlrich Sibiller2020-01-061-1/+1
| |
| * Keystroke.c: fix formatting of keystrokesUlrich Sibiller2020-01-061-1/+1
| | | | | | | | used the wrong macro for length determination
| * Init.h: Fix superfluous spaceUlrich Sibiller2020-01-061-1/+1
| |
| * Display.c: reformat some commentsUlrich Sibiller2020-01-061-39/+43
| |
| * Display.c: scope improvementsUlrich Sibiller2020-01-061-33/+23
| |
| * Reconnect.c: fix typosUlrich Sibiller2020-01-061-1/+1
| |
| * Keyboard.c: improve keyboard outputUlrich Sibiller2020-01-061-1/+1
| |
| * Error.c: mark NX changesUlrich Sibiller2020-01-061-19/+24
| | | | | | | | | | nxagentPrintError is derived from XlibInt.c:_XprintDefaultError. Let's add missing code and mark NX changes.
| * Keyboard.c: fix typoUlrich Sibiller2020-01-061-1/+1
| |
| * Remove the numVideoScreens xprintism.Ulrich Sibiller2020-01-062-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Backport of this xorg-xserver commit: commit a82e6efb7b9b2ab9a1597b002f375c5ee105e7f5 Author: Adam Jackson <ajax@redhat.com> Date: Wed Jul 16 14:19:04 2008 -0400 Remove the numVideoScreens xprintism. This was to account for cases where you had video and print screens in the same server. Lunacy. Leave the slot in ScreenInfo, but rename it, and stop looking at it.
| * nxagent: remove another xprint renmantUlrich Sibiller2020-01-067-35/+34
| | | | | | | | | | | | reqeuestingClient was dropped with the XPRINT drop. Unfortunetely we are usign it in Image.c. So let's introduce nxagentRequestingClient as replacement for now.
| * drop xprint remnants: xpstubsUlrich Sibiller2020-01-064-63/+0
| | | | | | | | | | | | | | | | | | | | Backport from xorg-xserver: commit 641ce9c706ce3cbf726fb3fc9623161bb96ba127 Author: George Sapountzis <gsap7@yahoo.gr> Date: Thu Jan 3 17:28:04 2008 +0200 drop xprint remnants: xpstubs
| * nxagent: drop CYGWIN and WIN32 definesUlrich Sibiller2020-01-064-82/+0
| |
| * Image.c: code simplification/scopeUlrich Sibiller2020-01-061-4/+2
| |
| * Screen.c: code simplification/scopeUlrich Sibiller2020-01-061-266/+185
| |
| * Screen.c: fix handling of NX_SHADOW atomUlrich Sibiller2020-01-061-10/+3
| | | | | | | | Needs to be an XlibAtom!
| * Screen.c: drop variableUlrich Sibiller2020-01-061-7/+3
| |
| * Window.c: code simplifications/scopeUlrich Sibiller2020-01-061-11/+5
| |
| * Window.c: fix commentUlrich Sibiller2020-01-061-2/+2
| |
| * Rootless.c: fix wordingUlrich Sibiller2020-01-061-1/+1
| |
| * X.Org bug 4947/Sun bug 6646626: Xv extension not byte-swapping properlyUlrich Sibiller2020-01-061-23/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes ArcticaProject/nx-libs#165 commit dfd682b582636a36345144bcf835e3ee46718d90 Author: Alan Coopersmith <alan.coopersmith@sun.com> Date: Wed Jan 2 19:27:22 2008 -0800 X.Org bug 4947/Sun bug 6646626: Xv extension not byte-swapping properly X.Org Bugzilla #4947 <https://bugs.freedesktop.org/show_bug.cgi?id=4947> Sun bug 6646626 <http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=6646626> Don't use swapped data after swapping it. When done swapping data, send the swapped data, not the address of the pointer to it, to the client.
| * NXproperty.c: simplify ChangeWindowPropertyUlrich Sibiller2020-01-062-110/+11
| |
| * Events.c: improve debugging outputUlrich Sibiller2020-01-061-7/+25
| | | | | | | | | | | | | | Extend nxagentHandleClientMessageEvent: better debug messages, suppress warning "Invalid window in ClientMessage" for legal case. I came across this while testing some clipboard stuff.
| * os: unifdef WIN32Ulrich Sibiller2020-01-0610-306/+6
| |
| * os: unifdef UNIXWAREUlrich Sibiller2020-01-061-1/+1
| |
| * os: unifdef __SCO__Ulrich Sibiller2020-01-064-62/+6
| |
| * os: unifdef ISCUlrich Sibiller2020-01-063-34/+5
| |