Page MenuHomeFreeBSD
Differential D13943

Cleanup kldxref sources
ClosedPublic
Actions

Authored by mhorne063_gmail.com on Jan 16 2018, 10:36 PM.
Tags
None
Referenced Files
F103807232: D13943.id38068.diff
Fri, Nov 29, 5:19 PM
Unknown Object (File)
Tue, Nov 26, 12:09 PM
Unknown Object (File)
Tue, Nov 26, 6:53 AM
Unknown Object (File)
Fri, Nov 22, 5:34 PM
Unknown Object (File)
Oct 2 2024, 6:17 AM
Unknown Object (File)
Sep 30 2024, 9:26 PM
Unknown Object (File)
Sep 20 2024, 3:12 AM
Unknown Object (File)
Sep 16 2024, 2:12 PM
View All 40 Files
Subscribers
andrew
imp

Details

Reviewers
emaste
cem
kib
Commits
rS328094: kldxref: improve style(9)
Summary

There are some style inconsistencies in the kldxref source files, this patch should fix most of them in accordance to style(9)
Some of these were discussed on a previous revision here.
Quick summary:

  • Use designated initializers for structs
  • Always use bracketed return style
  • No initialization in declarations
  • Align function prototype names
  • Remove old commented code/unused imports

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

mhorne063_gmail.com created this revision.Jan 16 2018, 10:36 PM
mhorne063_gmail.com added inline comments.Jan 16 2018, 10:47 PM
usr.sbin/kldxref/ef.c
121

The alignment here is correct, phabricator just doesn't seem to display it properly.

usr.sbin/kldxref/ef_obj.c
139

Same as above, alignment doesn't display properly in phabricator.

kib added inline comments.Jan 17 2018, 9:35 AM
usr.sbin/kldxref/ef.c
83

Can you extract the change of the return value to Elf_Off ?

296

Perhaps use %jx and (uintmax_t)ef_get_offset().

304

if (error != 0)

mhorne063_gmail.com updated this revision to Diff 38096.Jan 17 2018, 5:59 PM

Use explicit conditions (error != 0).
Remove the change to ef_get_offset() return type for another patch.

mhorne063_gmail.com marked 2 inline comments as done.Jan 17 2018, 6:00 PM
kib accepted this revision.Jan 17 2018, 7:00 PM
This revision is now accepted and ready to land.Jan 17 2018, 7:00 PM
mhorne063_gmail.com updated this revision to Diff 38104.Jan 17 2018, 7:50 PM

Changed pointer justification in the struct at the beginning of ef.c

This revision now requires review to proceed.Jan 17 2018, 7:50 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jan 17 2018, 8:00 PM
Closed by commit rS328094: kldxref: improve style(9) (authored by emaste). · Explain Why
This revision was automatically updated to reflect the committed changes.
Herald added subscribers: andrew, imp. · View Herald TranscriptJan 17 2018, 8:00 PM

Revision Contents
Changeset List

PathSize
usr.sbin/
kldxref/
182 lines
11 lines
106 lines
13 lines
86 lines

Diff 38068

View Options

usr.sbin/kldxref/ef.c

Loading...
View Options

usr.sbin/kldxref/ef_aarch64.c

Loading...
View Options

usr.sbin/kldxref/ef_obj.c

Loading...
View Options

usr.sbin/kldxref/ef_powerpc.c

Loading...
View Options

usr.sbin/kldxref/kldxref.c

Loading...