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

	F172890964: D44474.id136103.diff
	Mon, Sep 21, 9:30 PM

	F172890815: D44474.id136966.diff
	Mon, Sep 21, 9:29 PM

	F172890594: D44474.id137070.diff
	Mon, Sep 21, 9:27 PM

	F172872247: D44474.diff
	Mon, Sep 21, 6:12 PM

	Unknown Object (File)
	Sat, Sep 12, 9:45 AM

	Unknown Object (File)
	Thu, Sep 10, 6:58 PM

	Unknown Object (File)
	Thu, Sep 10, 11:01 AM

	Unknown Object (File)
	Tue, Sep 8, 3:17 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 56745
Build 53633: 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
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 136103

View Options

lib/libc/posix1e/acl_to_text.c

acl_to_text: include sys/param.h for MAXLOGNAMEClosedPublicActions