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/>
index | next in thread | previous in thread | raw e-mail
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=279303 --- 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 |= CE_BINARY; > break; > case 'c': > working->flags |= 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 :) -- You are receiving this mail because: You are the assignee for the bug.home | help
Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-279303-227-84o9UitSjW>