Date: Sun, 26 May 2024 23:31:09 +0000 From: bugzilla-noreply@freebsd.org To: bugs@FreeBSD.org Subject: [Bug 279303] usr.sbin/newsyslog: Fix case of the 'P' flag in newsyslog.conf's manpage Message-ID: <bug-279303-227-84o9UitSjW@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-279303-227@https.bugs.freebsd.org/bugzilla/> References: <bug-279303-227@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D279303 --- Comment #3 from Joshua Kinard <freebsd@kumba.dev> --- (In reply to Ed Maste from comment #2) The switch statement is changing the flag characters to lower case before it checks them: > for (; q && *q && !isspacech(*q); q++) { > switch (tolowerch(*q)) { > case 'b': > working->flags |=3D CE_BINARY; > break; > case 'c': > working->flags |=3D CE_CREATE; > break; In the man page, 'p' is the only config flag that's lowercase, which is why= I am assuming that it's the typo. If it's the other way around and lowercase= is supposed to be the correct format, then there's fourteen typos for the other uppercased flags that have gone unnoticed for a long time instead of one :) --=20 You are receiving this mail because: You are the assignee for the bug.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-279303-227-84o9UitSjW>