memchr.3: mention the len argument in the functions description
ClosedPublic
Actions

Authored by kib on Jul 19 2025, 12:18 PM.

Tags

None

Referenced Files

	F131765674: D51424.id.diff
	Sat, Oct 11, 12:03 AM

	F131765669: D51424.id158906.diff
	Sat, Oct 11, 12:03 AM

	F131765645: D51424.id158784.diff
	Sat, Oct 11, 12:02 AM

	F131765644: D51424.id158879.diff
	Sat, Oct 11, 12:02 AM

	F131731717: D51424.diff
	Fri, Oct 10, 5:37 PM

	Unknown Object (File)
	Wed, Oct 8, 12:07 AM

	Unknown Object (File)
	Fri, Sep 19, 5:37 AM

	Unknown Object (File)
	Fri, Sep 12, 2:44 AM

Subscribers

Details

Reviewers

emaste
pauamma_gundo.com

Group Reviewers

manpages

Commits

rG441995e161a2: memchr.3: mention the len argument in the functions description

Summary

Otherwise, its use is only specified in the error case, which is
somewhat obscure.  Also properly typeset NULL.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kib created this revision.Jul 19 2025, 12:18 PM

Herald added subscribers: ziaee, imp. · View Herald TranscriptJul 19 2025, 12:18 PM

kib requested review of this revision.Jul 19 2025, 12:18 PM

With or without removing "the", as appropriate.

lib/libc/string/memchr.3
68	Should "the" be here? I think it would read better without, but I'm not a native speaker. Fixable on commit if I'm correct.

This revision is now accepted and ready to land.Jul 19 2025, 9:52 PM

emaste added inline comments.Jul 21 2025, 3:56 PM

lib/libc/string/memchr.3
68	Probably "limited to the first .Fa len characters"

'limited to the first'

This revision now requires review to proceed.Jul 22 2025, 2:49 AM

This revision was not accepted when it landed; it landed in state Needs Review.Jul 22 2025, 3:11 PM

Closed by commit rG441995e161a2: memchr.3: mention the len argument in the functions description (authored by kib). · Explain Why

This revision was automatically updated to reflect the committed changes.

kib added a commit: rG441995e161a2: memchr.3: mention the len argument in the functions description.

Revision Contents
Changeset List

Path

Size

lib/

libc/

string/

memchr.3

16 lines

Diff 158906

View Options

memchr.3: mention the len argument in the functions descriptionClosedPublicActions