• Dmitri Shuralyov's avatar
    path/filepath: make Abs("") return working directory on Windows · 91f07c57
    Dmitri Shuralyov authored
    The current Abs docs say:
    
    	// If the path is not absolute it will be joined with the current
    	// working directory to turn it into an absolute path.
    
    The empty string is not an absolute path, so the docs suggest that the
    empty string should be joined with the current working directory to
    turn it into an absolute path. This was already the case on all
    platforms other than Windows. Per the decision in issue #24441,
    this change makes it work on Windows too.
    
    Since the empty string is not a valid path for the purposes of calling
    os.Stat on it, we can't simply add the empty string test case to
    absTests, which TestAbs uses. It would error when trying to do:
    
    	info, err := os.Stat(path)
    
    I didn't find a good way to modify TestAbs to handle this situation
    without significantly complicating its code and compromising the test.
    So, a separate test is created for testing Abs on empty string input.
    
    Fixes #24441.
    
    Change-Id: I11d8ae2f6e6e358f3e996372ee2a0449093898d2
    Reviewed-on: https://go-review.googlesource.com/112935Reviewed-by: 's avatarIan Lance Taylor <iant@golang.org>
    Reviewed-by: 's avatarAlex Brainman <alex.brainman@gmail.com>
    Run-TryBot: Ian Lance Taylor <iant@golang.org>
    TryBot-Result: Gobot Gobot <gobot@golang.org>
    91f07c57
Name
Last commit
Last update
..
filepath Loading commit data...
example_test.go Loading commit data...
match.go Loading commit data...
match_test.go Loading commit data...
path.go Loading commit data...
path_test.go Loading commit data...