Improve path processing (#131)

* Refs #20755: Normilize path Signed-off-by: elianalf <[email protected]> * Refs #20755: Substitute backslash with slash Signed-off-by: elianalf <[email protected]> --------- Signed-off-by: elianalf <[email protected]>
eProsima · Apr 26, 2024 · 60ec03c · 60ec03c
1 parent 7acf712
commit 60ec03c
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/src/main/java/com/eprosima/idl/context/Context.java b/src/main/java/com/eprosima/idl/context/Context.java
@@ -46,6 +46,7 @@
 import com.eprosima.idl.util.Util;
 import java.io.File;
 import java.io.StringReader;
+import java.nio.file.Paths;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
@@ -112,7 +113,7 @@ public Context(
 
         // The scope file has to be initialized because could occur the preprocessor
         // is not called (using -ppDisable).
-        m_scopeFile = m_file;
+        m_scopeFile = Paths.get(m_file).normalize().toString();
 
         m_includePaths = new ArrayList<String>();
         m_dependencies = new LinkedHashSet<String>();
@@ -139,12 +140,7 @@ public Context(
             {
                 include = include.substring(m_directoryFile.length());
             }
-            // Add last separator (can be empty by now...)
-            if (!include.isEmpty() && include.charAt(include.length() - 1) != java.io.File.separatorChar)
-            {
-                include += java.io.File.separator;
-            }
-            m_includePaths.add(include);
+            m_includePaths.add(Paths.get(include).normalize().toString() + java.io.File.separator);
         }
 
         // Reorder include paths;
@@ -886,7 +882,7 @@ public void processPreprocessorLine(
 
             // Read filename
             scanner.next();
-            String file = scanner.next();
+            String file = Paths.get(scanner.next()).normalize().toString();
 
             // Read flags.
             boolean systemFile = false, enteringFile = false, exitingFile = false;
@@ -986,6 +982,8 @@ else if (flag == 3)
                                     includeFile.substring(0, includeFile.length() - 4).replace('.', '_');
                                     // Change back to "." for relative paths.
                                     dependency_to_add = dependency_to_add.replaceAll("__/", "../");
+                                    // Substitute "\\" with File separator.
+                                    dependency_to_add = dependency_to_add.replace('\\', '/');
 
                                     m_directIncludeDependencies.add(dependency_to_add);
                                 }