Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 11 Nov 2009 12:10:04 GMT
From:      Bruce Evans <brde@optusnet.com.au>
To:        freebsd-bugs@FreeBSD.org
Subject:   Re: misc/140457: Grammar fix for isspace(3)
Message-ID:  <200911111210.nABCA4O9091941@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help
The following reply was made to PR misc/140457; it has been noted by GNATS.

From: Bruce Evans <brde@optusnet.com.au>
To: Jeremy Huddleston <jeremyhu@apple.com>
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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200911111210.nABCA4O9091941>