Unverified Commit 1d301c51 authored by Mihai Moldovan's avatar Mihai Moldovan

Merge branch 'sunweaver-pr/fix-compiler-warnings' into 3.6.x

parents 91ca51bf 8bb1579f
...@@ -897,7 +897,7 @@ static void PutEntry( ...@@ -897,7 +897,7 @@ static void PutEntry(
/* bump to loose table, if any */ /* bump to loose table, if any */
table = *(prev = &table->next); table = *(prev = &table->next);
if (!table || table->name != q || if (!table || table->name != q ||
!quarks[2] != table->leaf) (!quarks[2]) != table->leaf)
break; /* not found */ break; /* not found */
} }
} }
......
...@@ -1762,7 +1762,7 @@ static int __glXBindSwapBarrierSGIX(__GLXclientState *cl, GLbyte *pc) ...@@ -1762,7 +1762,7 @@ static int __glXBindSwapBarrierSGIX(__GLXclientState *cl, GLbyte *pc)
if (ret == Success) { if (ret == Success) {
if (barrier) if (barrier)
/* add source for cleanup when drawable is gone */ /* add source for cleanup when drawable is gone */
AddResource(drawable, __glXSwapBarrierRes, (void *)screen); AddResource(drawable, __glXSwapBarrierRes, (void *)(intptr_t)screen);
else else
/* delete source */ /* delete source */
FreeResourceByType(drawable, __glXSwapBarrierRes, FALSE); FreeResourceByType(drawable, __glXSwapBarrierRes, FALSE);
......
...@@ -113,7 +113,7 @@ __glXFBMemSwapBuffers(__GLXdrawablePrivate *glxPriv) ...@@ -113,7 +113,7 @@ __glXFBMemSwapBuffers(__GLXdrawablePrivate *glxPriv)
depth, depth,
0, 0, width, height, 0, 0, width, height,
pad, ZPixmap, pad, ZPixmap,
buf); (char *)buf);
return GL_TRUE; return GL_TRUE;
} }
......
...@@ -52,7 +52,7 @@ ...@@ -52,7 +52,7 @@
** Fetch a double from potentially unaligned memory. ** Fetch a double from potentially unaligned memory.
*/ */
#ifdef __GLX_ALIGN64 #ifdef __GLX_ALIGN64
#define __GLX_MEM_COPY(dst,src,n) if (src && dst) memcpy(dst,src,n) #define __GLX_MEM_COPY(dst,src,n) memmove(dst,src,n)
#define __GLX_GET_DOUBLE(dst,src) __GLX_MEM_COPY(&dst,src,8) #define __GLX_GET_DOUBLE(dst,src) __GLX_MEM_COPY(&dst,src,8)
#else #else
#define __GLX_GET_DOUBLE(dst,src) (dst) = *((GLdouble*)(src)) #define __GLX_GET_DOUBLE(dst,src) (dst) = *((GLdouble*)(src))
......
...@@ -358,9 +358,9 @@ ChangeKbdFeedback (client, dev, mask, k, f) ...@@ -358,9 +358,9 @@ ChangeKbdFeedback (client, dev, mask, k, f)
kctrl.autoRepeat = defaultKeyboardControl.autoRepeat; kctrl.autoRepeat = defaultKeyboardControl.autoRepeat;
else else
kctrl.autoRepeats[inx] &= ~kmask; kctrl.autoRepeats[inx] &= ~kmask;
kctrl.autoRepeats[inx] = kctrl.autoRepeats[inx] =
(kctrl.autoRepeats[inx] & ~kmask) | (kctrl.autoRepeats[inx] & ~kmask) |
(defaultKeyboardControl.autoRepeats[inx] & kmask); (defaultKeyboardControl.autoRepeats[inx] & kmask);
} }
else else
{ {
......
...@@ -4664,12 +4664,12 @@ void nxagentDumpInputDevicesState(void) ...@@ -4664,12 +4664,12 @@ void nxagentDumpInputDevicesState(void)
} }
} }
fprintf(stderr, "\nKeyboard device state: \n\tdevice [%p]\n\tlast grab time [%lu]" fprintf(stderr, "\nKeyboard device state: \n\tdevice [%p]\n\tlast grab time [%u]"
"\n\tfrozen [%s]\n\tstate [%s]\n\tother [%p]\n\tevent count [%d]" "\n\tfrozen [%s]\n\tstate [%s]\n\tother [%p]\n\tevent count [%d]"
"\n\tfrom passive grab [%s]\n\tactivating key [%d]", dev, "\n\tfrom passive grab [%s]\n\tactivating key [%d]", (void *)dev,
dev -> grabTime.milliseconds, dev -> sync.frozen ? "Yes": "No", dev -> grabTime.milliseconds, dev -> sync.frozen ? "Yes": "No",
nxagentGrabStateToString(dev -> sync.state), nxagentGrabStateToString(dev -> sync.state),
dev -> sync.other, dev -> sync.evcount, (void *)dev -> sync.other, dev -> sync.evcount,
dev -> fromPassiveGrab ? "Yes" : "No", dev -> fromPassiveGrab ? "Yes" : "No",
dev -> activatingKey); dev -> activatingKey);
...@@ -4679,7 +4679,7 @@ void nxagentDumpInputDevicesState(void) ...@@ -4679,7 +4679,7 @@ void nxagentDumpInputDevicesState(void)
{ {
fprintf(stderr, "\nKeyboard grab state: \n\twindow pointer [%p]" fprintf(stderr, "\nKeyboard grab state: \n\twindow pointer [%p]"
"\n\towner events flag [%s]\n\tgrab mode [%s]", "\n\towner events flag [%s]\n\tgrab mode [%s]",
grab -> window, grab -> ownerEvents ? "True" : "False", (void *)grab -> window, grab -> ownerEvents ? "True" : "False",
grab -> keyboardMode ? "asynchronous" : "synchronous"); grab -> keyboardMode ? "asynchronous" : "synchronous");
/* /*
...@@ -4693,8 +4693,8 @@ void nxagentDumpInputDevicesState(void) ...@@ -4693,8 +4693,8 @@ void nxagentDumpInputDevicesState(void)
{ {
fprintf(stderr, "\nPassive grab state: \n\tdevice [%p]\n\towner events flag [%s]" fprintf(stderr, "\nPassive grab state: \n\tdevice [%p]\n\towner events flag [%s]"
"\n\tpointer grab mode [%s]\n\tkeyboard grab mode [%s]\n\tevent type [%d]" "\n\tpointer grab mode [%s]\n\tkeyboard grab mode [%s]\n\tevent type [%d]"
"\n\tmodifiers [%x]\n\tbutton/key [%u]\n\tevent mask [%lx]", "\n\tmodifiers [%x]\n\tbutton/key [%u]\n\tevent mask [%x]",
grab -> device, grab -> ownerEvents ? "True" : "False", (void *)grab -> device, grab -> ownerEvents ? "True" : "False",
grab -> pointerMode ? "asynchronous" : "synchronous", grab -> pointerMode ? "asynchronous" : "synchronous",
grab -> keyboardMode ? "asynchronous" : "synchronous", grab -> keyboardMode ? "asynchronous" : "synchronous",
grab -> type, grab -> modifiersDetail.exact, grab -> type, grab -> modifiersDetail.exact,
...@@ -4724,12 +4724,12 @@ void nxagentDumpInputDevicesState(void) ...@@ -4724,12 +4724,12 @@ void nxagentDumpInputDevicesState(void)
} }
} }
fprintf(stderr, "\nPointer device state: \n\tdevice [%p]\n\tlast grab time [%lu]" fprintf(stderr, "\nPointer device state: \n\tdevice [%p]\n\tlast grab time [%u]"
"\n\tfrozen [%s]\n\tstate [%s]\n\tother [%p]\n\tevent count [%d]" "\n\tfrozen [%s]\n\tstate [%s]\n\tother [%p]\n\tevent count [%d]"
"\n\tfrom passive grab [%s]\n\tactivating button [%d]", dev, "\n\tfrom passive grab [%s]\n\tactivating button [%d]", (void *)dev,
dev -> grabTime.milliseconds, dev -> sync.frozen ? "Yes" : "No", dev -> grabTime.milliseconds, dev -> sync.frozen ? "Yes" : "No",
nxagentGrabStateToString(dev -> sync.state), nxagentGrabStateToString(dev -> sync.state),
dev -> sync.other, dev -> sync.evcount, (void *)dev -> sync.other, dev -> sync.evcount,
dev -> fromPassiveGrab ? "Yes" : "No", dev -> fromPassiveGrab ? "Yes" : "No",
dev -> activatingKey); dev -> activatingKey);
...@@ -4739,7 +4739,7 @@ void nxagentDumpInputDevicesState(void) ...@@ -4739,7 +4739,7 @@ void nxagentDumpInputDevicesState(void)
{ {
fprintf(stderr, "\nPointer grab state: \n\twindow pointer [%p]" fprintf(stderr, "\nPointer grab state: \n\twindow pointer [%p]"
"\n\towner events flag [%s]\n\tgrab mode [%s]", "\n\towner events flag [%s]\n\tgrab mode [%s]",
grab -> window, grab -> ownerEvents ? "True" : "False", (void *)grab -> window, grab -> ownerEvents ? "True" : "False",
grab -> pointerMode ? "asynchronous" : "synchronous"); grab -> pointerMode ? "asynchronous" : "synchronous");
if (grab -> window != pWin) if (grab -> window != pWin)
...@@ -4754,8 +4754,8 @@ void nxagentDumpInputDevicesState(void) ...@@ -4754,8 +4754,8 @@ void nxagentDumpInputDevicesState(void)
{ {
fprintf(stderr, "\nPassive grab state: \n\tdevice [%p]\n\towner events flag [%s]" fprintf(stderr, "\nPassive grab state: \n\tdevice [%p]\n\towner events flag [%s]"
"\n\tpointer grab mode [%s]\n\tkeyboard grab mode [%s]\n\tevent type [%d]" "\n\tpointer grab mode [%s]\n\tkeyboard grab mode [%s]\n\tevent type [%d]"
"\n\tmodifiers [%x]\n\tbutton/key [%u]\n\tevent mask [%lx]", "\n\tmodifiers [%x]\n\tbutton/key [%u]\n\tevent mask [%x]",
grab -> device, grab -> ownerEvents ? "True" : "False", (void *)grab -> device, grab -> ownerEvents ? "True" : "False",
grab -> pointerMode ? "asynchronous" : "synchronous", grab -> pointerMode ? "asynchronous" : "synchronous",
grab -> keyboardMode ? "asynchronous" : "synchronous", grab -> keyboardMode ? "asynchronous" : "synchronous",
grab -> type, grab -> modifiersDetail.exact, grab -> type, grab -> modifiersDetail.exact,
......
...@@ -309,7 +309,7 @@ Reply Total Cached Bits In Bits Out Bits/Reply Ratio ...@@ -309,7 +309,7 @@ Reply Total Cached Bits In Bits Out Bits/Reply Ratio
if (!(dispatchException & DE_TERMINATE)) if (!(dispatchException & DE_TERMINATE))
dispatchException = 0; dispatchException = 0;
while (!dispatchException) while (!dispatchException)
{ {
if (*icheck[0] != *icheck[1]) if (*icheck[0] != *icheck[1])
{ {
......
...@@ -95,16 +95,20 @@ ProcXvDispatch(ClientPtr client) ...@@ -95,16 +95,20 @@ ProcXvDispatch(ClientPtr client)
result = (XineramaXvPutVideo(client)); result = (XineramaXvPutVideo(client));
else else
#endif #endif
result = (ProcXvPutVideo(client)); {
break; result = (ProcXvPutVideo(client));
}
break;
case xv_PutStill: case xv_PutStill:
#ifdef PANORAMIX #ifdef PANORAMIX
if(!noPanoramiXExtension) if(!noPanoramiXExtension)
result = (XineramaXvPutStill(client)); result = (XineramaXvPutStill(client));
else else
#endif #endif
result = (ProcXvPutStill(client)); {
break; result = (ProcXvPutStill(client));
}
break;
case xv_GetVideo: result = (ProcXvGetVideo(client)); break; case xv_GetVideo: result = (ProcXvGetVideo(client)); break;
case xv_GetStill: result = (ProcXvGetStill(client)); break; case xv_GetStill: result = (ProcXvGetStill(client)); break;
case xv_GrabPort: result = (ProcXvGrabPort(client)); break; case xv_GrabPort: result = (ProcXvGrabPort(client)); break;
...@@ -117,16 +121,20 @@ ProcXvDispatch(ClientPtr client) ...@@ -117,16 +121,20 @@ ProcXvDispatch(ClientPtr client)
result = (XineramaXvStopVideo(client)); result = (XineramaXvStopVideo(client));
else else
#endif #endif
{
result = (ProcXvStopVideo(client)); result = (ProcXvStopVideo(client));
break; }
break;
case xv_SetPortAttribute: case xv_SetPortAttribute:
#ifdef PANORAMIX #ifdef PANORAMIX
if(!noPanoramiXExtension) if(!noPanoramiXExtension)
result = (XineramaXvSetPortAttribute(client)); result = (XineramaXvSetPortAttribute(client));
else else
#endif #endif
{
result = (ProcXvSetPortAttribute(client)); result = (ProcXvSetPortAttribute(client));
break; }
break;
case xv_GetPortAttribute: result = (ProcXvGetPortAttribute(client)); break; case xv_GetPortAttribute: result = (ProcXvGetPortAttribute(client)); break;
case xv_QueryBestSize: result = (ProcXvQueryBestSize(client)); break; case xv_QueryBestSize: result = (ProcXvQueryBestSize(client)); break;
case xv_QueryPortAttributes: result = (ProcXvQueryPortAttributes(client)); break; case xv_QueryPortAttributes: result = (ProcXvQueryPortAttributes(client)); break;
...@@ -136,8 +144,10 @@ ProcXvDispatch(ClientPtr client) ...@@ -136,8 +144,10 @@ ProcXvDispatch(ClientPtr client)
result = (XineramaXvPutImage(client)); result = (XineramaXvPutImage(client));
else else
#endif #endif
{
result = (ProcXvPutImage(client)); result = (ProcXvPutImage(client));
break; }
break;
#ifdef MITSHM #ifdef MITSHM
case xv_ShmPutImage: case xv_ShmPutImage:
#ifdef PANORAMIX #ifdef PANORAMIX
...@@ -145,8 +155,10 @@ ProcXvDispatch(ClientPtr client) ...@@ -145,8 +155,10 @@ ProcXvDispatch(ClientPtr client)
result = (XineramaXvShmPutImage(client)); result = (XineramaXvShmPutImage(client));
else else
#endif #endif
{
result = (ProcXvShmPutImage(client)); result = (ProcXvShmPutImage(client));
break; }
break;
#endif #endif
case xv_QueryImageAttributes: result = (ProcXvQueryImageAttributes(client)); break; case xv_QueryImageAttributes: result = (ProcXvQueryImageAttributes(client)); break;
case xv_ListImageFormats: result = (ProcXvListImageFormats(client)); break; case xv_ListImageFormats: result = (ProcXvListImageFormats(client)); break;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment