Repository: maven-surefire
Updated Branches:
  refs/heads/SUREFIRE-1376_javadoc [created] 558615e9a


[SUREFIRE-1376] improved javadoc for Win "long path" and escaping

Project: http://git-wip-us.apache.org/repos/asf/maven-surefire/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven-surefire/commit/558615e9
Tree: http://git-wip-us.apache.org/repos/asf/maven-surefire/tree/558615e9
Diff: http://git-wip-us.apache.org/repos/asf/maven-surefire/diff/558615e9

Branch: refs/heads/SUREFIRE-1376_javadoc
Commit: 558615e9a0010b8c41750445af1e31687a5b1b92
Parents: 9c77aa4
Author: Hervé Boutemy <hbout...@apache.org>
Authored: Mon Jun 5 11:29:50 2017 +0200
Committer: Hervé Boutemy <hbout...@apache.org>
Committed: Mon Jun 5 11:29:50 2017 +0200

----------------------------------------------------------------------
 .../maven/plugin/surefire/SurefireHelper.java   | 24 ++++++++++----------
 1 file changed, 12 insertions(+), 12 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/558615e9/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/SurefireHelper.java
----------------------------------------------------------------------
diff --git 
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/SurefireHelper.java
 
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/SurefireHelper.java
index cd49249..a356803 100644
--- 
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/SurefireHelper.java
+++ 
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/SurefireHelper.java
@@ -57,15 +57,13 @@ public final class SurefireHelper
     public static final String DUMPSTREAM_FILENAME_FORMATTER = 
DUMP_FILE_PREFIX + "%d" + DUMPSTREAM_FILE_EXT;
 
     /**
-     * see sun/nio/fs/WindowsPath <br>
+     * The maximum path that does not require long path prefix on Windows.<p>
+     * see sun/nio/fs/WindowsPath
      * 
http://hg.openjdk.java.net/jdk8/jdk8/jdk/file/7534523b4174/src/windows/classes/sun/nio/fs/WindowsPath.java#l46
-     * https://msdn.microsoft.com/en-us/library/aa365247(VS.85).aspx#maxpath
-     * <br>
-     * <br>
-     * The maximum path that does not require long path prefix. On Windows <br>
-     * the maximum path is 260 minus 1 (NUL) but for directories it is 260 <br>
-     * minus 12 minus 1 (to allow for the creation of a 8.3 file in the <br>
-     * directory).
+     * <br>and 
https://msdn.microsoft.com/en-us/library/aa365247(VS.85).aspx#maxpath
+     * <p>
+     * The maximum path is 260 minus 1 (NUL) but for directories it is 260
+     * minus 12 minus 1 (to allow for the creation of a 8.3 file in the 
directory).
      */
     private static final int MAX_PATH_LENGTH_WINDOWS = 247;
 
@@ -180,11 +178,13 @@ public final class SurefireHelper
     }
 
     /**
-     * Normalized file path for Windows; otherwise returns {@code path}.
-     * <br>
+     * Escape file path for Windows when the path is too long; otherwise 
returns {@code path}.
+     * <p>See <a href="
      * 
http://hg.openjdk.java.net/jdk8/jdk8/jdk/file/7534523b4174/src/windows/classes/sun/nio/fs/WindowsPath.java#l46
-     * <br>
-     * https://msdn.microsoft.com/en-us/library/aa365247(VS.85).aspx#maxpath
+     * ">sun/nio/fs/WindowsPath</a> for "long path" value explanation (=247), 
and 
+     * <a 
href="https://msdn.microsoft.com/en-us/library/aa365247(VS.85).aspx#maxpath">MSDN
 article</a>
+     * for detailed escaping strategy explanation: in short, {@code \\?\} 
prefix for path with drive letter
+     * or {@code \\?\UNC\} for UNC path
      *
      * @param path    source path
      * @return escaped to platform path

Reply via email to