From: "Ulrich Weigand" <uweigand@de.ibm.com>
To: eliz@gnu.org
Cc: gdb-patches@sourceware.org
Subject: Re: [rfc v2][4/6] Readlink as file I/O target operation
Date: Mon, 16 Jan 2012 13:29:00 -0000 [thread overview]
Message-ID: <201201161232.q0GCWN33024751@d06av02.portsmouth.uk.ibm.com> (raw)
In-Reply-To: <83hazzjw3x.fsf@gnu.org> from "Eli Zaretskii" at Jan 13, 2012 08:28:02 PM
Eli Zaretskii wrote:
> > Date: Fri, 13 Jan 2012 19:15:12 +0100 (CET)
> > From: "Ulrich Weigand" <uweigand@de.ibm.com>
> > @@ -36193,6 +36197,16 @@ error occurred.
> > Delete the file at @var{pathname} on the target. Return 0,
> > or -1 if an error occurs. @var{pathname} is a string.
> >
> > +@item vFile:readlink: @var{pathname}
> > +Read value of symbolic link @var{pathname} on the target. Return
> > +the number of bytes read, or -1 if an error occurs.
>
> This part is okay, but please don't use "pathname" when you really
> mean "file name". GNU Coding Standards frown on using "path" or its
> derivatives for anything but PATH-style directory lists.
I'll be happy to use "filename" instead, but the currently existing
packets (open, unlink) also use "pathname" today. Should those be
changed to "filename" too?
(B.t.w. note that those packets are directly related to the corresponding
POSIX routines open/unlink/readlink -- the documentation of those routines,
whether in POSIX itself or in the corresponding Linux man pages consistently
refers to those arguments as "path" or "pathname" ... I'm wondering whether
it is a deliberate decision on the part of the GNU Coding Standards to deviate
from established terminology in that area?)
Bye,
Ulrich
--
Dr. Ulrich Weigand
GNU Toolchain for Linux on System z and Cell BE
Ulrich.Weigand@de.ibm.com
next prev parent reply other threads:[~2012-01-16 12:32 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-13 18:16 Ulrich Weigand
2012-01-13 18:29 ` Eli Zaretskii
2012-01-16 13:29 ` Ulrich Weigand [this message]
2012-01-16 14:48 ` Eli Zaretskii
2012-01-16 15:03 ` Ulrich Weigand
2012-01-16 17:18 ` Eli Zaretskii
2012-01-16 15:37 ` Pedro Alves
2012-01-16 17:26 ` Ulrich Weigand
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201201161232.q0GCWN33024751@d06av02.portsmouth.uk.ibm.com \
--to=uweigand@de.ibm.com \
--cc=eliz@gnu.org \
--cc=gdb-patches@sourceware.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