Fix error found by new clang operator precendence warning
ClosedPublic
Actions

Authored by arichardson on Nov 17 2019, 5:27 PM.

Tags

None

Referenced Files

	F133108492: D22433.id64509.diff
	Thu, Oct 23, 1:24 AM

	F133106100: D22433.id64509.diff
	Thu, Oct 23, 12:54 AM

	F133106099: D22433.id64501.diff
	Thu, Oct 23, 12:54 AM

	F133106092: D22433.id.diff
	Thu, Oct 23, 12:54 AM

	F133071568: D22433.diff
	Wed, Oct 22, 5:20 PM

	Unknown Object (File)
	Mon, Oct 13, 5:32 AM

	Unknown Object (File)
	Sun, Oct 12, 4:34 PM

	Unknown Object (File)
	Sep 12 2025, 1:33 PM

View All 81 Files

Subscribers

imp

Details

Reviewers

• ian

Commits

rS360317: MFC r354796 (by arichardson):
rS354796: Fix error found by new clang operator precendence warning

Summary

error: operator '?:' has lower precedence than '|'; '|' will be evaluated first

I discovered this in CheriBSD after updating our fork of clang to the latest
upstream master.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 27608
Build 25823: arc lint + arc unit

Event Timeline

arichardson created this revision.Nov 17 2019, 5:27 PM

Harbormaster completed remote builds in B27608: Diff 64501.Nov 17 2019, 5:27 PM

• ian accepted this revision.Nov 17 2019, 6:00 PM

This revision is now accepted and ready to land.Nov 17 2019, 6:00 PM

Closed by commit rS354796: Fix error found by new clang operator precendence warning (authored by arichardson). · Explain WhyNov 17 2019, 7:04 PM

This revision was automatically updated to reflect the committed changes.

arichardson added a commit: rS354796: Fix error found by new clang operator precendence warning.

Herald added a subscriber: imp. · View Herald TranscriptNov 17 2019, 7:04 PM

dim added a commit: rS360317: MFC r354796 (by arichardson):.Apr 25 2020, 3:46 PM

Revision Contents
Changeset List

Path

Size

usr.sbin/

i2c/

i2c.c

2 lines

Diff 64501

View Options

Fix error found by new clang operator precendence warningClosedPublicActions