A bug in the code, but maybe too hard to fix. Might need a documentation fix instead.

Has this been updated yet?

Has this been updated yet?

No, otherwise the Gerrit change request would be linked with this issue.

There is actually a test case for path.Join which joins two empty strings and checks the output is an empty string. So not sure if it is a bug.

For most cases this is true, but for the case of only empty filenames, it isn't. There, path.Join("") returns an empty string while path.Clean("") returns dot.

That is because path.Clean is called only for non-empty strings. Which is what the documentation tries to say. But I agree this part ; in particular, all empty strings are ignored. is not clear that Clean is being called after ignoring all empty strings. I believe updating the docs is the safest recourse.

@shivi28 - would you like to work on this ?

@agnivade I can work on this unless @shivi28 already has or was planning to.

There hasn't been a response, so feel free to take it up.

Change https://golang.org/cl/194339 mentions this issue: path: inprove documentation to call out cases where Clean is called

@erutherford, thanks for the fix; could you also fix filepath.Join documentation which has the same problem?

I noticed you removed "all empty strings are ignored", is that ok?
Without that sentence how can I be sure that the result of path.Join("", "foo") is "foo" and not "/foo"?

EDIT: I'm opening a new issue