> Thus, I propose the following respin: implement xdirname and xbasename
> using a strlcat/strlcpy semantic. This is more natural in most calls anyway.

Good idea.  Being used to the strlfoo functions, it looks a bit strange
to make xplen the middlemost argument, but I'm just being me now. :-)

> +     if (*dp && dp[0] == '/' && dp[1] == '\0') {
Is the *dp check necessary?  I realize it's like this in the old code
too, but it caught my attention now.

In any case, the diff looks good, and I found no problems while
testing it.

Reply via email to