From owner-freebsd-bugs@FreeBSD.ORG Wed Nov 11 12:10:04 2009 Return-Path: Delivered-To: freebsd-bugs@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C10921065676 for ; Wed, 11 Nov 2009 12:10:04 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 976AF8FC16 for ; Wed, 11 Nov 2009 12:10:04 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.3/8.14.3) with ESMTP id nABCA4q7091942 for ; Wed, 11 Nov 2009 12:10:04 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.3/8.14.3/Submit) id nABCA4O9091941; Wed, 11 Nov 2009 12:10:04 GMT (envelope-from gnats) Date: Wed, 11 Nov 2009 12:10:04 GMT Message-Id: <200911111210.nABCA4O9091941@freefall.freebsd.org> To: freebsd-bugs@FreeBSD.org From: Bruce Evans Cc: Subject: Re: misc/140457: Grammar fix for isspace(3) X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Bruce Evans List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 11 Nov 2009 12:10:04 -0000 The following reply was made to PR misc/140457; it has been noted by GNATS. From: Bruce Evans To: Jeremy Huddleston Cc: freebsd-gnats-submit@FreeBSD.org, freebsd-bugs@FreeBSD.org Subject: Re: misc/140457: Grammar fix for isspace(3) Date: Wed, 11 Nov 2009 23:08:28 +1100 (EST) On Tue, 10 Nov 2009, Jeremy Huddleston wrote: >> Fix: > --- isspace.3.bsdnew 2009-11-10 13:13:10.000000000 -0800 > +++ isspace.3 2009-11-10 13:45:41.000000000 -0800 > @@ -54,9 +54,9 @@ For any locale, this includes the follow > .It "\&``\et''\t``\en''\t``\ev''\t``\ef''\t``\er''\t`` ''" > .El > .Pp > -In the "C" locale > +In the "C" locale, This and the other commas you want to add are improvements, but is this because they are required by grammar or style? > .Fn isspace > -successful test is limited to this characters only. > +successful test is limited to these characters only. > The value of the argument must be representable as an > .Vt "unsigned char" > or the value of isspace.3 has several more grammar errors. In FreeBSD changes, I noticed: % Index: isspace.3 % =================================================================== % RCS file: /home/ncvs/src/lib/libc/locale/isspace.3,v % retrieving revision 1.1 % retrieving revision 1.22 % diff -u -2 -r1.1 -r1.22 % --- isspace.3 27 May 1994 04:56:44 -0000 1.1 % +++ isspace.3 4 Sep 2009 07:44:58 -0000 1.22 % ... % .Sh DESCRIPTION % The % .Fn isspace % -function tests for the standard white-space characters. % -In the ASCII character set, this includes the following characters: % -.sp % -.Bl -column \&000_``0''__ \&000_``0''__ \&000_``0''__ \&000_``0''__ \&000_``0''__ % -.It \&011\ ht \t012\ nl \t013\ vt \t014\ np \t015\ cr % -.It \&040\ sp % +function tests for the white-space characters. This got mangled by removing "standard" without removing "the". The latter applies to "standard", not the the white-space characters, and not removing it defeats the point of removing "standard" (by suggesting that there is a single set of white-space characters, instead of a standard set and nonstandard extensions). % +For any locale, this includes the following standard characters: % +.Pp % +.Bl -column \&`\et''___ \&``\et''___ \&``\et''___ \&``\et''___ \&``\et''___ \&``\et''___ % +.It "\&``\et''\t``\en''\t``\ev''\t``\ef''\t``\er''\t`` ''" % .El % +.Pp % +In the "C" locale % +.Fn isspace % +successful test is limited to this characters only. "isspace successful test" still has a couple of grammar errors. The C standard has much better wording for this, of course. POSIX.1 (which FreeBSD has permission to copy) unfortunately doesn't copy the C standard, and it has grammatically correct but otherwise poor wording (at least in a draft 2001 version) (it refers to the C standard and a generic section), so copying it wouldn't be useful. The C standard says "standard white space" like the above used to, but doesn't say "the standard", and it handles the bug fixed in the above better by saying "standard ... or ... locale-specific...". Then it lists the standard white-space characters (hopefully the same set as above). Then it has very good wording instead of "successful test is limited to this [sic] characters". isblank.3 has the same bugs. Bruce