hashirama/xserver
Watch
0
0
Fork
You've already forked xserver
 0
mirror of https://github.com/X11Libre/xserver.git synced 2026-10-26 01:59:57 +00:00
Code Issues Projects Releases Wiki Activity

[PR #3570] Handle -Wimplicit-fallthrough warnings from gcc 16.1

Browse source 
Gets rid of 36 -Wimplicit-fallthrough warnings

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/2272>
PR: https://github.com/X11Libre/xserver/pull/3570
This commit is contained in:
Alan Coopersmith 2026-08-09 14:03:19 -07:00 • committed by Enrico Weigelt, metux IT consult
commit 952b268ed0
8 changed files with 24 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

5
Xext/xkeyboard/ddxBeep.c
View file

@ -139,6 +139,7 @@ _XkbDDXBeepExpire(OsTimerPtr timer, CARD32 now, void *arg)
switch (xkbInfo->beepType) {
default:
ErrorF("[xkb] Unknown beep type %d\n", xkbInfo->beepType);
/* fallthrough */
case _BEEP_NONE:
duration = 0;
break;
@ -207,6 +208,7 @@ _XkbDDXBeepExpire(OsTimerPtr timer, CARD32 now, void *arg)
case _BEEP_LED_CHANGE:
if (name == None)
name = ledChange;
/* fallthrough */
case _BEEP_FEATURE_CHANGE:
if (name == None)
name = featureChange;
@ -237,9 +239,11 @@ _XkbDDXBeepExpire(OsTimerPtr timer, CARD32 now, void *arg)
case _BEEP_SLOW_PRESS:
if (name == None)
name = slowPress;
/* fallthrough */
case _BEEP_SLOW_ACCEPT:
if (name == None)
name = slowAccept;
/* fallthrough */
case _BEEP_SLOW_RELEASE:
if (name == None)
name = slowRelease;
@ -249,6 +253,7 @@ _XkbDDXBeepExpire(OsTimerPtr timer, CARD32 now, void *arg)
case _BEEP_BOUNCE_REJECT:
if (name == None)
name = bounceReject;
/* fallthrough */
case _BEEP_SLOW_REJECT:
if (name == None)
name = slowReject;

5
Xext/xtest/xtest.c
View file

@ -308,14 +308,19 @@ ProcXTestFakeInput(ClientPtr client)
switch (dv->num_valuators) {
case 6:
valuators[base + 5] = dv->valuator5;
/* fallthrough */
case 5:
valuators[base + 4] = dv->valuator4;
/* fallthrough */
case 4:
valuators[base + 3] = dv->valuator3;
/* fallthrough */
case 3:
valuators[base + 2] = dv->valuator2;
/* fallthrough */
case 2:
valuators[base + 1] = dv->valuator1;
/* fallthrough */
case 1:
valuators[base] = dv->valuator0;
break;

7
hw/kdrive/src/kinput.c
View file

@ -423,18 +423,25 @@ KdPointerProc(DeviceIntPtr pDevice, int onoff)
default:
case 7:
btn_labels[6] = XIGetKnownProperty(BTN_LABEL_PROP_BTN_HWHEEL_RIGHT);
/* fallthrough */
case 6:
btn_labels[5] = XIGetKnownProperty(BTN_LABEL_PROP_BTN_HWHEEL_LEFT);
/* fallthrough */
case 5:
btn_labels[4] = XIGetKnownProperty(BTN_LABEL_PROP_BTN_WHEEL_DOWN);
/* fallthrough */
case 4:
btn_labels[3] = XIGetKnownProperty(BTN_LABEL_PROP_BTN_WHEEL_UP);
/* fallthrough */
case 3:
btn_labels[2] = XIGetKnownProperty(BTN_LABEL_PROP_BTN_RIGHT);
/* fallthrough */
case 2:
btn_labels[1] = XIGetKnownProperty(BTN_LABEL_PROP_BTN_MIDDLE);
/* fallthrough */
case 1:
btn_labels[0] = XIGetKnownProperty(BTN_LABEL_PROP_BTN_LEFT);
/* fallthrough */
case 0:
break;
}

2
hw/xfree86/common/xf86fbman.c
View file

@ -586,6 +586,7 @@ localQueryLargestOffscreenArea(ScreenPtr pScreen,
pbox = RegionRects(newRegion);
break;
}
/* fallthrough */
case 1:
if (offman->NumUsedAreas) {
FBLinkPtr pLink;
@ -608,6 +609,7 @@ localQueryLargestOffscreenArea(ScreenPtr pScreen,
pbox = RegionRects(newRegion);
break;
}
/* fallthrough */
default:
nbox = RegionNumRects(offman->FreeBoxes);
pbox = RegionRects(offman->FreeBoxes);

2
hw/xfree86/drivers/video/modesetting/dri2.c
View file

@ -652,7 +652,7 @@ ms_dri2_frame_event_handler(uint64_t msc,
ms_dri2_exchange_buffers(drawable, frame_info->front, frame_info->back);
break;
}
/* else fall through to blit */
/* else fall through */
case MS_DRI2_QUEUE_SWAP:
ms_dri2_blit_swap(drawable, frame_info->front, frame_info->back);
DRI2SwapComplete(frame_info->client, drawable, msc, tv_sec, tv_usec,

2
hw/xfree86/parser/Flags.c
View file

@ -112,11 +112,13 @@ xf86parseFlagsSection(XF86ConfFlagsPtr ptr)
*/
case DEFAULTLAYOUT:
strvalue = TRUE;
/* fallthrough */
case BLANKTIME:
case STANDBYTIME:
case SUSPENDTIME:
case OFFTIME:
hasvalue = TRUE;
/* fallthrough */
case DONTZAP:
case DONTZOOM:
case DISABLEVIDMODE:

1
include/fb.h
View file

@ -194,6 +194,7 @@ typedef int FbStride;
break; \
case sizeof (FbBits) - 3: \
FbStorePart(dst,sizeof (FbBits) - 3,CARD8,xor); \
/* fallthrough */ \
case sizeof (FbBits) - 2: \
FbStorePart(dst,sizeof (FbBits) - 2,CARD16,xor); \
break; \

1
mi/miarc.c
View file

@ -1434,6 +1434,7 @@ miArcJoin(DrawablePtr pDraw, GCPtr pGC, miArcFacePtr pLeft,
polyLen = 5;
break;
}
/* fallthrough */
case JoinBevel:
poly[0] = corner;
poly[1] = center;