Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH v2 2/4] compat/basename: make basename() conform to POSIX

From: Junio C Hamano <hidden>
Date: 2016-06-15 23:07:41

Johannes Schindelin [off-list ref] writes:
Hi Junio,

On Fri, 8 Jan 2016, Junio C Hamano wrote:
quoted
Johannes Schindelin [off-list ref] writes:
quoted
According to POSIX, basename("/path/") should return "path", not
"path/". Likewise, basename(NULL) and basename("abc") should both
return ".".
Did you mean basename("abc"), not basename(""), here?  
I don't understand: I wrote basename("abc")... ;-)
Yeah, I read that.  What I didn't read was you wrote 'path' between
slashes in the first example (the MUA was trying to render it as
"path" in italic or something silly like that), and made me
confused: If 'path' has to go to 'path', what's so special about
'abc' to make it go '.'?

Upon second reading I noticed that slashes around the first one, but
forgot to remove the "why 'abc'" comment.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help