Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 30 Apr 2010 00:01:23 -0700
From:      Xin LI <delphij@gmail.com>
To:        Ben Kaduk <minimarmot@gmail.com>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Xin LI <delphij@freebsd.org>
Subject:   Re: svn commit: r207283 - head/usr.bin/gzip
Message-ID:  <w2oa78074951004300001l9047051ejf3a924f94b49da38@mail.gmail.com>
In-Reply-To: <q2w47d0403c1004271055yec72d9d5g734e2bce3b6cae21@mail.gmail.com>
References:  <201004271750.o3RHoi8G078883@svn.freebsd.org> <q2w47d0403c1004271055yec72d9d5g734e2bce3b6cae21@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
Hi, Ben,

On Tue, Apr 27, 2010 at 10:55 AM, Ben Kaduk <minimarmot@gmail.com> wrote:
> On Tue, Apr 27, 2010 at 1:50 PM, Xin LI <delphij@freebsd.org> wrote:
>> Author: delphij
>> Date: Tue Apr 27 17:50:43 2010
>> New Revision: 207283
>> URL: http://svn.freebsd.org/changeset/base/207283
>>
>> Log:
>> =C2=A0Language improvements to make the BUGS section easier to read.
>>
>> =C2=A0Reviewed by: =C2=A0Alexander Best <alexbestms wwu de>
>> =C2=A0MFC after: =C2=A0 =C2=A013 days
>>
>> Modified:
>> =C2=A0head/usr.bin/gzip/gzip.1
>>
>> Modified: head/usr.bin/gzip/gzip.1
>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D
>> --- head/usr.bin/gzip/gzip.1 =C2=A0 =C2=A0Tue Apr 27 15:59:38 2010 =C2=
=A0 =C2=A0 =C2=A0 =C2=A0(r207282)
>> +++ head/usr.bin/gzip/gzip.1 =C2=A0 =C2=A0Tue Apr 27 17:50:43 2010 =C2=
=A0 =C2=A0 =C2=A0 =C2=A0(r207283)
>> @@ -218,8 +218,8 @@ with unpack support written by
>> =C2=A0.An Xin LI Aq delphij@FreeBSD.org .
>> =C2=A0.Sh BUGS
>> =C2=A0According to RFC 1952, the recorded file size is stored in a 32-bi=
t
>> -integer and therefore it can not represent files that is bigger than
>> -4GB in size. =C2=A0This limitation also applies to
>> +integer, therefore, it can not represent files larger than 4GB.
>> +This limitation also applies to
>
> Still not quite right -- the old version was wrong because it had
> "files that is" instead of "files that are". =C2=A0The new version has a
> comma splice, though -- the comma in "32-bit integer, therefore"
> should be replaced with either a semicolon or a full stop.
>
> I would probably also put in two "the"s in the text following this
> change -- "the -l option" and "the gzip utility".

I would appreciate some native English speaking committers to help me
on this, I personally have no much strong opinion on the wording :-/

>> =C2=A0.Fl l
>> =C2=A0option of
>> =C2=A0.Nm
>> _______________________________________________
>> svn-src-head@freebsd.org mailing list
>> http://lists.freebsd.org/mailman/listinfo/svn-src-head
>> To unsubscribe, send any mail to "svn-src-head-unsubscribe@freebsd.org"
>>
>



--=20
Xin LI <delphij@delphij.net> http://www.delphij.net



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