acl_to_text: include sys/param.h for MAXLOGNAME
ClosedPublic
Actions

Authored by brooks on Mar 22 2024, 8:26 PM.

Details

Reviewers

kib
imp
trasz

Commits

rG418b4c2a80a2: acl_to_text: include sys/param.h for MAXLOGNAME

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

brooks created this revision.Mar 22 2024, 8:26 PM

Herald added a subscriber: imp. · View Herald TranscriptMar 22 2024, 8:26 PM

brooks requested review of this revision.Mar 22 2024, 8:26 PM

Harbormaster completed remote builds in B56745: Diff 136103.Mar 22 2024, 8:26 PM

brooks added a child revision: D44468: sys/acl.h: reduce header pollution.Mar 22 2024, 8:27 PM

brooks added reviewers: kib, imp, trasz.

kib added inline comments.Mar 23 2024, 6:57 AM

lib/libc/posix1e/acl_to_text.c
32–33	Since sys/param.h includes so many other stuff which might better be namespaced, I suggest to add namespace.h before its inclusion.

namespace sys/param.h

Harbormaster completed remote builds in B57067: Diff 136966.Fri, Apr 12, 9:48 PM

kib accepted this revision.Fri, Apr 12, 11:33 PM

This revision is now accepted and ready to land.Fri, Apr 12, 11:33 PM

Closed by commit rG418b4c2a80a2: acl_to_text: include sys/param.h for MAXLOGNAME (authored by brooks). · Explain WhyMon, Apr 15, 8:44 PM

This revision was automatically updated to reflect the committed changes.

brooks added a commit: rG418b4c2a80a2: acl_to_text: include sys/param.h for MAXLOGNAME.

Revision Contents
Changeset List

Path

Size

lib/

libc/

posix1e/

acl_to_text.c

2 lines

Diff 137070

View Options

lib/libc/posix1e/acl_to_text.c

acl_to_text: include sys/param.h for MAXLOGNAMEClosedPublicActions