Page MenuHomeFreeBSD
Differential D39520

bootpd: Use C89 function definitions.
ClosedPublic
Actions

Authored by jhb on Apr 12 2023, 3:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 31, 8:21 AM
Unknown Object (File)
Sat, Oct 25, 8:46 PM
Unknown Object (File)
Sat, Oct 25, 8:46 PM
Unknown Object (File)
Oct 19 2025, 1:01 AM
Unknown Object (File)
Oct 11 2025, 4:43 AM
Unknown Object (File)
Oct 11 2025, 12:14 AM
Unknown Object (File)
Sep 9 2025, 7:53 PM
Unknown Object (File)
Sep 9 2025, 5:16 AM
View All Files
Subscribers
imp

Details

Reviewers
emaste
Commits
rG8b356c888167: bootpd: Use C89 function definitions.
Summary

Trim a few duplicate (but incomplete) function prototypes as well.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 50851
Build 47742: arc lint + arc unit

Event Timeline

jhb created this revision.Apr 12 2023, 3:32 PM
Herald added a subscriber: imp. · View Herald TranscriptApr 12 2023, 3:32 PM
jhb requested review of this revision.Apr 12 2023, 3:32 PM
Harbormaster completed remote builds in B50851: Diff 120199.Apr 12 2023, 3:32 PM
jhb added a parent revision: D39519: smbfs: Use C89 function definition for cf_getopt..Apr 12 2023, 3:32 PM
jhb added a child revision: D39521: rpc.rstatd/rwalld: Use more accurate function pointer types..
emaste accepted this revision.Apr 12 2023, 3:58 PM
emaste added inline comments.
libexec/bootpd/dovend.c
39

perhaps we can change these to uint8_t in a subsequent commit?

libexec/bootpd/report.c
38

this seems kind of odd (unrelated to your work)

This revision is now accepted and ready to land.Apr 12 2023, 3:58 PM
jhb added inline comments.Apr 12 2023, 6:57 PM
libexec/bootpd/dovend.c
39

This entire codebase is in a different style from CMU. I tried to just focus on fixing the function definitions and leaving the rest alone.

libexec/bootpd/report.c
38

There's a lot of oddness in this code. :/

Closed by commit rG8b356c888167: bootpd: Use C89 function definitions. (authored by jhb). · Explain WhyApr 18 2023, 6:32 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rG8b356c888167: bootpd: Use C89 function definitions..

Revision Contents
Changeset List

PathSize
libexec/
bootpd/
28 lines
bootpgw/
4 lines
2 lines
20 lines
18 lines
4 lines
7 lines
50 lines
21 lines
15 lines
95 lines
5 lines
10 lines
tools/
bootpef/
9 lines
bootptest/
13 lines
21 lines

Diff 120199

View Options

libexec/bootpd/bootpd.c

Loading...
View Options

libexec/bootpd/bootpgw/bootpgw.c

Loading...
View Options

libexec/bootpd/dovend.h

Loading...
View Options

libexec/bootpd/dovend.c

Loading...
View Options

libexec/bootpd/dumptab.c

Loading...
View Options

libexec/bootpd/getether.c

Loading...
View Options

libexec/bootpd/getif.c

Loading...
View Options

libexec/bootpd/hash.c

Loading...
View Options

libexec/bootpd/hwaddr.c

Loading...
View Options

libexec/bootpd/lookup.c

Loading...
View Options

libexec/bootpd/readfile.c

Loading...
View Options

libexec/bootpd/report.c

Loading...
View Options

libexec/bootpd/rtmsg.c

Loading...
View Options

libexec/bootpd/tools/bootpef/bootpef.c

Loading...
View Options

libexec/bootpd/tools/bootptest/bootptest.c

Loading...
View Options

libexec/bootpd/tools/bootptest/print-bootp.c

Loading...