Page MenuHomeFreeBSD
Differential D18577

ixgbe: this statement may fall through warnings with gcc
ClosedPublic
Actions

Authored by tsoome on Dec 16 2018, 9:58 AM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 18 2024, 11:28 AM
Unknown Object (File)
Oct 22 2024, 4:58 PM
Unknown Object (File)
Oct 21 2024, 4:07 PM
Unknown Object (File)
Oct 17 2024, 9:33 AM
Unknown Object (File)
Oct 10 2024, 4:09 PM
Unknown Object (File)
Oct 10 2024, 4:09 PM
Unknown Object (File)
Oct 10 2024, 4:08 PM
Unknown Object (File)
Oct 10 2024, 4:04 PM
View All 61 Files
Subscribers
ae
imp

Details

Reviewers
sbruno
Commits
rS344094: MFC: r343203
rS344095: MFC: r343203
rS343203: ixgbe: this statement may fall through warnings with gcc
Summary

The recent gcc versions (7 and 8 at least) can check for switch case
statements for fall through (implicit-fallthrough). When fall through
is intentional, the default method for warning suppression is to place
comment /* FALLTHROUGH */ exactly before next case statement.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tsoome created this revision.Dec 16 2018, 9:58 AM
Herald added a reviewer: sbruno. · View Herald TranscriptDec 16 2018, 9:58 AM
Herald added subscribers: ae, imp. · View Herald Transcript
Harbormaster completed remote builds in B21597: Diff 52075.Dec 16 2018, 9:58 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jan 19 2019, 8:45 PM
Closed by commit rS343203: ixgbe: this statement may fall through warnings with gcc (authored by tsoome). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
ixgbe/
4 lines
6 lines

Diff 53031

View Options

head/sys/dev/ixgbe/ixgbe_82599.c

Loading...
View Options

head/sys/dev/ixgbe/ixgbe_common.c

Loading...