Page MenuHomeFreeBSD
Differential D34199

IPMI: Fixing whitespace
Needs ReviewPublic
Actions

Authored by david.elliott3040_gmail.com on Feb 8 2022, 12:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jun 13, 10:59 AM
Unknown Object (File)
Wed, Jun 10, 11:36 PM
Unknown Object (File)
Tue, Jun 9, 1:47 PM
Unknown Object (File)
Sat, May 30, 2:54 PM
Unknown Object (File)
Fri, May 29, 5:36 PM
Unknown Object (File)
Thu, May 28, 4:55 AM
Unknown Object (File)
May 20 2026, 2:26 AM
Unknown Object (File)
May 18 2026, 3:11 PM
View All 88 Files
Subscribers
imp

Details

Reviewers
mav
Summary

Erroneous whitespace in ipmivars.h

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

david.elliott3040_gmail.com created this revision.Feb 8 2022, 12:25 PM
Herald added a subscriber: imp. · View Herald TranscriptFeb 8 2022, 12:25 PM
david.elliott3040_gmail.com requested review of this revision.Feb 8 2022, 12:25 PM
david.elliott3040_gmail.com retitled this revision from Fixing whitespace to IPMI: Fixing whitespace.Feb 8 2022, 12:44 PM
david.elliott3040_gmail.com added a reviewer: mav.Feb 9 2022, 2:16 PM
mav added a comment.Feb 11 2022, 12:40 AM

I don't think it is erroneous. I think it was intentional to highlight different meaning of those constants. Such practice is often used when register addresses and values are listed together. Though yes, additional indentation is usually done not before #define, but after it.

Revision Contents
Changeset List

PathSize
sys/
dev/
ipmi/
10 lines

Diff 102487

View Options

sys/dev/ipmi/ipmivars.h

Loading...