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)
Sep 11 2025, 3:45 AM
Unknown Object (File)
Sep 9 2025, 8:37 PM
Unknown Object (File)
Sep 8 2025, 5:46 PM
Unknown Object (File)
Sep 3 2025, 12:31 AM
Unknown Object (File)
Aug 20 2025, 11:16 PM
Unknown Object (File)
Aug 19 2025, 1:11 AM
Unknown Object (File)
Jul 28 2025, 9:05 PM
Unknown Object (File)
Jun 29 2025, 5:56 PM
View All 56 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...