New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

delete redundancy check for path at DeleteDirectory and DiskUsage #855

Open

baobaoyeye wants to merge 3 commits into baidu:master from baobaoyeye:del_redundancy

Contributor

baobaoyeye commented Feb 20, 2017

（1）NameSpace::NormalizePath 这个函数会默认加上 “/”因此这两个验证都是冗余的，
（2）因为DeleteDirectory的时候如果传一个空字符串就把“/”都删掉太坑，因此提到NormalizePath前面

baobaoyeye added 3 commits

February 20, 2017 19:34


          delete redundancy check for path at DeleteDirectory and DiskUsage

cfdc347


          delete redundancy check for path at DeleteDirectory and DiskUsage

23aa1b1


          delete redundancy check for path at DeleteDirectory and DiskUsage

0c04a74

yvxiang approved these changes

View reviewed changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet