jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Gaul <notificati...@github.com>
Subject Re: [jclouds/jclouds] Ensure that jclouds filesystem provider consistently uses / (#1024)
Date Fri, 21 Oct 2016 22:34:55 GMT
andrewgaul commented on this pull request.



> @@ -786,10 +782,11 @@ private String removeFileSeparatorFromBorders(String pathToBeCleaned,
boolean on
 
       // search for separator chars
       if (!onlyTrailing) {
-         if (pathToBeCleaned.substring(0, 1).equals(File.separator))
+         if (pathToBeCleaned.substring(0, 1).equals("/") || pathToBeCleaned.substring(0,
1).equals("\\"))

Call `charAt` instead?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1024#pullrequestreview-5330256
Mime
View raw message