removed redundant Clean

shuai-z · shuai-z · commit ce407ccff4fb · 2014-10-26T13:55:29.000+08:00
The doc (or src) says: The result is Cleaned. http://golang.org/pkg/path/filepath/#Join Signed-off-by: shuai-z <zs.broccoli@gmail.com>
diff --git a/pkg/symlink/fs.go b/pkg/symlink/fs.go
@@ -35,7 +35,6 @@ func FollowSymlinkInScope(link, root string) (string, error) {
 
 	for _, p := range strings.Split(link, "/") {
 		prev = filepath.Join(prev, p)
-		prev = filepath.Clean(prev)
 
 		loopCounter := 0
 		for {
@@ -72,7 +71,7 @@ func FollowSymlinkInScope(link, root string) (string, error) {
 				} else {
 					prev, _ = filepath.Abs(prev)
 
-					if prev = filepath.Clean(filepath.Join(filepath.Dir(prev), dest)); len(prev) < len(root) {
+					if prev = filepath.Join(filepath.Dir(prev), dest); len(prev) < len(root) {
 						prev = filepath.Join(root, filepath.Base(dest))
 					}
 				}

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,6 @@ func FollowSymlinkInScope(link, root string) (string, error) {`
`35`	`35`
`36`	`36`	`for _, p := range strings.Split(link, "/") {`
`37`	`37`	`prev = filepath.Join(prev, p)`
`38`		`- prev = filepath.Clean(prev)`
`39`	`38`
`40`	`39`	`loopCounter := 0`
`41`	`40`	`for {`
`@@ -72,7 +71,7 @@ func FollowSymlinkInScope(link, root string) (string, error) {`
`72`	`71`	`} else {`
`73`	`72`	`prev, _ = filepath.Abs(prev)`
`74`	`73`
`75`		`- if prev = filepath.Clean(filepath.Join(filepath.Dir(prev), dest)); len(prev) < len(root) {`
	`74`	`+ if prev = filepath.Join(filepath.Dir(prev), dest); len(prev) < len(root) {`
`76`	`75`	`prev = filepath.Join(root, filepath.Base(dest))`
`77`	`76`	`}`
`78`	`77`	`}`