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
F163258761: D18577.id53031.diff
Tue, Jul 21, 12:31 PM
Unknown Object (File)
Mon, Jul 20, 12:15 AM
Unknown Object (File)
Sat, Jul 18, 12:40 PM
Unknown Object (File)
Thu, Jul 16, 11:52 PM
Unknown Object (File)
Fri, Jul 3, 8:32 AM
Unknown Object (File)
Fri, Jul 3, 6:32 AM
Unknown Object (File)
Thu, Jul 2, 11:14 PM
Unknown Object (File)
Thu, Jul 2, 7:54 PM
View All 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...