Page MenuHomeFreeBSD

hpwmc_amd.c: style improvements
ClosedPublic

Authored by mhorne on Aug 1 2023, 4:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 29, 11:24 PM
Unknown Object (File)
Sat, Sep 27, 2:43 AM
Unknown Object (File)
Sun, Sep 21, 12:44 AM
Unknown Object (File)
Aug 26 2025, 11:26 PM
Unknown Object (File)
Aug 21 2025, 11:42 AM
Unknown Object (File)
Aug 14 2025, 11:02 PM
Unknown Object (File)
Jul 29 2025, 2:41 AM
Unknown Object (File)
Jul 28 2025, 4:55 PM
Subscribers

Details

Summary
  • Return style
  • Explicit value checks
  • Whitespace formatting
  • Comment formatting
  • Local variable declaration order
  • __unused annotations

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mhorne requested review of this revision.Aug 1 2023, 4:23 PM
sys/dev/hwpmc/hwpmc_amd.c
883

This snuck in; fixed locally.

jkoshy added inline comments.
sys/dev/hwpmc/hwpmc_amd.c
981

Add a comment about this being the __i386__ case?

This revision is now accepted and ready to land.Aug 3 2023, 6:00 PM
sys/dev/hwpmc/hwpmc_amd.c
981

The ifdef gets removed in D41274, so I really have no idea why I bothered touching these lines. I'll drop this hunk of the diff before pushing.

This revision was automatically updated to reflect the committed changes.