[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: doc cleanups and questions
From: |
Alain Magloire |
Subject: |
Re: doc cleanups and questions |
Date: |
Sat, 30 Dec 2000 22:56:24 -0500 (EST) |
Sorry Shaleh, I'll have to be brief:
> In several places the statement -- "The return value is @code{0} on success
> and
> a code number on error conditions" -- is made. 'a code number' just does not
> sound right. error code, something else please.
Right.
> the sfrom example, status is used to check numerous return values, but in some
> cases the value of status is not used:
>
> if ((status = foo()) != 0)
>
> why use status here instead of if (foo() != 0)?
don't know, probably a typo or something, Remember this was the early
drafts after our long IRC talks.
> the main info menu bullets need attention. Several are flagged as 'foo API'
> while others just say 'foo', i.e. Attributes.
Noted.
> why does the stream api example show how to make a web browser? (or use a goto
> for that matter (-:
8-) I will not be drawn in to a "goto is evil war" but maybe the example
was ill conceived as a "tutorial" example.
> attached is a patch to the doc dir. It cleans up much of the text to
> reasonable English. Alain (I assume it was you) you need to watch your tenses
> and numbers. Most of the patch is making verbs and nouns agree. Things like
Guilty on all charges, for me you people talk backward, I still do
not understand why you say "a black car" instead of "a car black".
> -The ressources allocate for @var{hdr} are freed.
> +The resources allocated for @var{hdr} are freed.
>
> plural noun, verb should agree.
Noted,
> --_=XFMail.1.4.6.Linux:20001229194512:336=_
> Content-Disposition: attachment; filename="mailutils.shaleh.patch"
??
I thought you had write access to the repository, Actually I'm
not sure where is the repository now, Jeff is making some heavy
changes, there will be a transition between subversions.gnu.org
and sourceforge.
Thanks for proof reading, it will be a good base to start
the documentation.
I got some heavy changes to commit(support for IMAP4)
and at last I'll be able to do.
% ./from
% MAIL=pop://localhost ./from
% MAIL=imap://localhost ./from
from the same mailutils/from/from.c code ;-)
Later.