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

	F132425031: D44474.id136966.diff
	Thu, Oct 16, 8:05 PM

	Unknown Object (File)
	Wed, Oct 15, 11:38 PM

	Unknown Object (File)
	Wed, Oct 15, 11:18 PM

	Unknown Object (File)
	Wed, Oct 15, 11:18 PM

	Unknown Object (File)
	Wed, Oct 15, 11:18 PM

	Unknown Object (File)
	Wed, Oct 15, 11:18 PM

	Unknown Object (File)
	Wed, Oct 15, 12:53 PM

	Unknown Object (File)
	Wed, Oct 1, 1:24 PM

View All 88 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 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.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 137070

View Options

lib/libc/posix1e/acl_to_text.c

acl_to_text: include sys/param.h for MAXLOGNAMEClosedPublicActions