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

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

Tags

None

Referenced Files

	F151694514: D44474.id136103.diff
	Fri, Apr 10, 2:02 AM

	F151638396: D44474.id136103.diff
	Thu, Apr 9, 4:55 PM

	F151637980: D44474.id136103.diff
	Thu, Apr 9, 4:50 PM

	Unknown Object (File)
	Tue, Apr 7, 12:43 PM

	Unknown Object (File)
	Mon, Apr 6, 9:11 AM

	Unknown Object (File)
	Sun, Apr 5, 10:58 AM

	Unknown Object (File)
	Thu, Mar 19, 12:45 AM

	Unknown Object (File)
	Fri, Mar 13, 10:01 PM

View All Files

Subscribers

imp

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 Passed

Unit

No Test Coverage

Build Status

Buildable 57067
Build 53955: arc lint + arc unit

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.Apr 12 2024, 9:48 PM

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

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

Closed by commit rG418b4c2a80a2: acl_to_text: include sys/param.h for MAXLOGNAME (authored by brooks). · Explain WhyApr 15 2024, 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 136966

View Options

lib/libc/posix1e/acl_to_text.c

acl_to_text: include sys/param.h for MAXLOGNAMEClosedPublicActions