Re: Hello fdclose

From: John Baldwin <jhb_at_freebsd.org>
Date: Thu, 20 Mar 2014 10:21:51 -0400
On Wednesday, March 19, 2014 4:28:15 pm Warren Block wrote:
> On Wed, 19 Mar 2014, John Baldwin wrote:
> 
> > On Wednesday, March 19, 2014 12:38:57 am Warren Block wrote:
> >> On Tue, 18 Mar 2014, John Baldwin wrote:
> >>
> >>> On Monday, March 17, 2014 7:23:19 pm Mariusz Zaborski wrote:
> >>>> Hi,
> >>>>
> >>>> After our previous discuss  [1] I prepare fdclosedir(3) function which
> >>>> was committed by Pawel (cc'ed) in commit r254499.
> >>>>
> >>>> A while ago I also prepare the fdclose function. Unfortunately, this
> >>>> new function is a little bit more tricky then previous one. Can I ask
> >>>> you for a review of this patch?
> >>>
> >>> I think the code is fine.  I have a few suggestions on the manpage wording:
> >>>
> >>> The
> >>> +.Fn fdclose
> >>> +function is equivalent to the
> >>> +.Fn fclose
> >>> +function except that this function returns file descriptor instead of
> >>> +closing it.
> >>> +.Pp
> >>> +The
> >>>
> >>> I would move fdclose() to its own paragraph and reword this sentence as:
> >>>
> >>>  "The fdclose() function is equivalent to fclose() except that it does
> >>>   not close the underlying file descriptor."
> >>
> >> .Fn fdclose
> >> is equivalent to
> >> .Fn fclose ,
> >> but the file descriptor is returned rather than closed.
> >>
> >> Likewise in other sections, the markup is supposed to do the job of
> >> pointing out that something is a function.
> >
> > Yes, but this has the 'no capital letter at the start of a sentence' problem.
> 
> I've heard that mentioned before, but have never seen any actual rule 
> regarding it.

All of my rules for that come from elementary school. :)

-- 
John Baldwin
Received on Thu Mar 20 2014 - 13:22:22 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:47 UTC