Differential D20427

ips(4): add missing braces
ClosedPublic
Actions

Authored by lwhsu on May 27 2019, 2:50 PM.

Tags

None

Referenced Files

	F169431442: D20427.id58051.diff
	Tue, Sep 1, 12:57 PM

	Unknown Object (File)
	Mon, Aug 31, 11:10 AM

	Unknown Object (File)
	Mon, Aug 31, 10:52 AM

	Unknown Object (File)
	Mon, Aug 31, 6:32 AM

	Unknown Object (File)
	Mon, Aug 31, 1:55 AM

	Unknown Object (File)
	Sat, Aug 29, 5:07 AM

	Unknown Object (File)
	Sat, Aug 29, 1:57 AM

	Unknown Object (File)
	Wed, Aug 26, 2:57 PM

View All Files

Subscribers

imp

Details

Reviewers

scottl

Commits

rS360297: MFC r348366 (by lwhsu):
rS348366: Add the likely missing braces in ips(4). This is found by gcc warning that

Summary

Add the very likely missing braces in ips(4). This is found by gcc warning that
the code is not guarded by the if clause and has misleading indentation.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

lwhsu created this revision.May 27 2019, 2:50 PM

Herald added a subscriber: imp. · View Herald TranscriptMay 27 2019, 2:50 PM

Harbormaster completed remote builds in B24536: Diff 57954.May 27 2019, 2:50 PM

lwhsu added a reviewer: scottl.May 27 2019, 2:51 PM

lwhsu removed a subscriber: scottl.

scottl accepted this revision.May 27 2019, 11:04 PM

This revision is now accepted and ready to land.May 27 2019, 11:04 PM

Closed by commit rS348366: Add the likely missing braces in ips(4). This is found by gcc warning that (authored by lwhsu). · Explain WhyMay 29 2019, 6:11 PM

This revision was automatically updated to reflect the committed changes.

dim added a commit: rS360297: MFC r348366 (by lwhsu):.Apr 25 2020, 12:39 PM

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

ips/

ips.c

3 lines

Diff 58051

View Options

ips(4): add missing bracesClosedPublicActions