Page MenuHomeFreeBSD

syslogd: Move struct definitions into header
AbandonedPublic

Authored by jfree on Aug 8 2023, 4:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 24, 2:56 AM
Unknown Object (File)
Tue, Apr 22, 1:19 PM
Unknown Object (File)
Mon, Apr 21, 2:48 AM
Unknown Object (File)
Fri, Apr 18, 4:23 PM
Unknown Object (File)
Mar 24 2025, 2:27 AM
Unknown Object (File)
Mar 23 2025, 1:58 PM
Unknown Object (File)
Mar 16 2025, 11:55 AM
Unknown Object (File)
Feb 20 2025, 4:25 AM
Subscribers

Details

Reviewers
markj
emaste
Summary
Extract the struct definitions from the source syslogd.c file and move
them into an isolated syslogd.h header file.

This change facilitates the compartmentalization of syslogd into
separate files.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

I'm ok with this, a general comment is just that we should seek to minimize syslogd.h, i.e., try to ensure it only contains stuff that really needs to be shared. For example, I'm pretty sure casper doesn't need RCVBUF_MINSIZE?

This revision is now accepted and ready to land.Aug 11 2023, 3:02 PM

I'm ok with this, a general comment is just that we should seek to minimize syslogd.h, i.e., try to ensure it only contains stuff that really needs to be shared. For example, I'm pretty sure casper doesn't need RCVBUF_MINSIZE?

A later patch will split syslogd into multiple files for configuration parsing, message parsing, and log parsing. This is all needed there.