From: ludovic.courtes@laas.fr (Ludovic Courtès)
Cc: guile-devel@gnu.org
Subject: Re: Socket API improvement, patch #6
Date: Wed, 02 Nov 2005 11:50:06 +0100 [thread overview]
Message-ID: <8764rbz5lt.fsf@laas.fr> (raw)
In-Reply-To: <87br196yd8.fsf@zip.com.au> (Kevin Ryde's message of "Sat, 29 Oct 2005 09:00:19 +1000")
Hi,
Kevin Ryde <user42@zip.com.au> writes:
> Beaut, I gave it a bit of a tweak and applied it.
Ok, I'm nitpicking: basically, for function descriptions, I prefer the
style that I used rather than your style. For instance, I suggested:
@deftypefn {C Function} {struct sockaddr *}scm_c_make_socket_address (SCM family, SCM address, SCM args, size_t *address_size)
Return a newly-allocated @code{sockaddr} structure that reflects
@var{address}, an address of family @var{family}, with the
family-specific parameters @var{args} (see the description of
@var{make-socket-address} for details). On success, a non-@code{NULL}
pointer is returned and @var{address_size} is updated to the actual
size (in bytes) of the returned address. The returned structure must
eventually be freed using @code{free ()}.
@end deftypefn
while you preferred:
@deftypefn {C Function} {struct sockaddr *} scm_c_make_socket_address (SCM family, SCM address, SCM args, size_t *outsize)
Return a newly-@code{malloc}ed @code{struct sockaddr} created from
arguments like those taken by @code{scm_make_socket_address} above.
The size (in bytes) of the @code{struct sockaddr} return is stored
into @code{*@var{outsize}}. An application must call @code{free} to
release the returned structure when no longer required.
@end deftypefn
IMO, the style you used is "sploppier" and less concise. The "formal"
declarative style I used is also more consistent with the rest of the
manual and other GNU manuals as well I think.
Ludovic.
_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/guile-devel
next prev parent reply other threads:[~2005-11-02 10:50 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <87oecutxox.fsf@laas.fr>
[not found] ` <87vf58cxxq.fsf@zagadka.de>
[not found] ` <87k6kwopv5.fsf@laas.fr>
[not found] ` <87fysk7ady.fsf@zagadka.de>
[not found] ` <87mzmpmcm2.fsf@laas.fr>
[not found] ` <87aci6u6f4.fsf@laas.fr>
[not found] ` <87psr22c2p.fsf@zip.com.au>
2005-09-22 13:57 ` [PATCH] Test-suite update Ludovic Courtès
2005-09-22 19:25 ` Neil Jerram
2005-09-22 19:41 ` Neil Jerram
2005-09-23 9:54 ` Ludovic Courtès
2005-09-23 18:57 ` Neil Jerram
2005-09-26 8:49 ` Ludovic Courtès
2005-09-26 23:15 ` Kevin Ryde
2005-09-22 21:15 ` Kevin Ryde
2005-09-22 22:06 ` Kevin Ryde
2005-09-23 7:43 ` Ludovic Courtès
2005-09-23 23:54 ` Kevin Ryde
2005-09-26 8:35 ` Ludovic Courtès
2005-09-26 23:18 ` Kevin Ryde
[not found] ` <87irwtqkop.fsf@laas.fr>
[not found] ` <87slvog9sd.fsf@zip.com.au>
[not found] ` <87wtkt9xyq.fsf_-_@laas.fr>
[not found] ` <87hdbg4dl7.fsf@laas.fr>
[not found] ` <87br1nakge.fsf@zip.com.au>
[not found] ` <87hdbfnu9n.fsf@laas.fr>
[not found] ` <87d5m2twaf.fsf@uni-dortmund.de>
[not found] ` <878xwjb123.fsf@laas.fr>
[not found] ` <874q76h9rh.fsf@zip.com.au>
[not found] ` <87hdb5qogp.fsf@zagadka.de>
[not found] ` <87zmovn4y5.fsf@zip.com.au>
[not found] ` <87fyqn1gzy.fsf@laas.fr>
[not found] ` <87br196yd8.fsf@zip.com.au>
2005-11-02 10:50 ` Ludovic Courtès [this message]
2005-11-02 20:07 ` Socket API improvement, patch #6 Kevin Ryde
2005-11-03 9:00 ` Ludovic Courtès
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/guile/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8764rbz5lt.fsf@laas.fr \
--to=ludovic.courtes@laas.fr \
--cc=guile-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).