Fixed PR comments

Author: kwin

doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DocumentRenderingContext.java

@@ -62,7 +62,7 @@ public class DocumentRenderingContext {
     /** The project's build directory, may be {@code null} rendered from a Doxia source) */
     private final File rootDirectory;
 
-    /** The site's root directory (never null), must be within below {@link #rootDirectory}, may be {@code null} rendered from a Doxia source */
+    /** The site's root directory, must be below {@link #rootDirectory}, may be {@code null} if not rendered from a Doxia source */
     private final File siteRootDirectory;
 
     /** optional descriptive text of the plugin which generated the output (usually Maven coordinates). Only set when document is not based on a Doxia source. */
@@ -315,7 +315,7 @@ public String getParserId() {
     }
 
     /**
-     * Get the relative path of the parent folder to site root.
+     * Get the relative path of the parent directory of this document to site root.
      *
      * @return the relative path to site root
      */

doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/SiteRenderingContext.java

@@ -23,7 +23,7 @@
 import java.util.Collection;
 import java.util.Collections;
 import java.util.Date;
-import java.util.HashSet;
+import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
@@ -46,15 +46,15 @@ public static class SiteDirectory {
         private File path;
         private boolean editable;
         private boolean skipDuplicates;
-        private final Collection<File> editableSourceDirectories = new HashSet<>();
+        private final Collection<File> editableSourceDirectories = new LinkedHashSet<>();
 
         public SiteDirectory(File path, boolean editable) {
             this(path, editable, false);
         }
 
         /**
          *
-         * @param path absolute path to the site directory containing doxia sources files, expected to have a Doxia Site layout, i.e. one directory per Doxia parser module
+         * @param path path to the site directory containing Doxia sources files, expected to have a Doxia Site layout, i.e. one directory per Doxia parser module
          * @param editable {@code true} if the site directory is expected to be editable
          * @param skipDuplicates flag indicating if duplicates in this directory should be skipped ({@code true}) or lead to an exception ({@code false})
          * @since 2.1
@@ -104,7 +104,7 @@ public Collection<File> getEditableSourceDirectories() {
             if (editableSourceDirectories.isEmpty()) {
                 return Collections.singleton(path);
             } else {
-                return editableSourceDirectories;
+                return Collections.unmodifiableCollection(editableSourceDirectories);
             }
         }
     }