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)
Sun, Nov 16, 1:45 AM
Unknown Object (File)
Mon, Nov 3, 8:05 PM
Unknown Object (File)
Oct 25 2025, 1:31 AM
Unknown Object (File)
Oct 23 2025, 6:04 AM
Unknown Object (File)
Oct 23 2025, 6:04 AM
Unknown Object (File)
Oct 22 2025, 9:56 PM
Unknown Object (File)
Oct 18 2025, 5:54 PM
Unknown Object (File)
Sep 11 2025, 3:45 AM
View All 63 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...