mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* RE: Re: [PATCH] use S_ISDIR() in link_path_walk() to decide whether the last path component is a directory
@ 2004-09-13 22:47 David Dabbs
  0 siblings, 0 replies; only message in thread
From: David Dabbs @ 2004-09-13 22:47 UTC (permalink / raw)
  To: 'ReiserFS List', linux-kernel; +Cc: viro


> viro wrote:
>>     if (*name == '/') {
>>        if (*(name+1)=='/' && *(name+2)==':') {
>>           name+=3;
>
>	Pathname resolution is a hell of a fundamental thing and kludges
>like that are too ugly to be acceptable.  If you can't make that clean
>and have to resort to stuffing "special cases" (read: barfbag of ioctl
>magnitude) into the areas that might be unspecified by POSIX, don't do it
>at all.
>

Even though SuS allows for implementation-specific resolution for pathnames
starting with "//"? It's kludgy, and I suspected that might be the response,
but I thought I'd float it nonetheless.

>I don't like the amount of handwaving from Hans, but *that* is far
>worse.  Vetoed.

Kludgy, yes, but far worse?  At least I bothered to take the SuS into
consideration and took the time to try an approach, however kludgy, that
might work within them. 

Bilious or not, thanks for the feedback.


david



^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2004-09-13 22:49 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-09-13 22:47 Re: [PATCH] use S_ISDIR() in link_path_walk() to decide whether the last path component is a directory David Dabbs

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome