bug-mailutils
[Top][All Lists]
Advanced

[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.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]