commit b4060b6e53944a7c3bdc5e62b288e7293a87652a
author nharmata <nharmata@google.com> Tue Apr 04 17:11:39 2017 +0000
committer Damien Martin-Guillerez <dmarting@google.com> Wed Apr 05 15:18:20 2017 +0200
tree 59b0f1f3d3e8e99412e060bb98b5a37fe90d9b6e
parent 3ac77cb94a4cf1bd1993a97fe79f2005b2b1a711

Refactor all ctor callsites of PathFragment to instead call a static 'create' method.

This paves the way for changing PathFragment to e.g. an abstract class with multiple subclasses. This way we can split out the windows-specific stuff into one of these concrete classes, making the code more readable and also saving memory (since the shallow heap size of the NonWindowsPathFragment subclass will hopefully be smaller than that of the current PathFragment).

This also lets us pursue gc churn optimizations. We can now do interning in PathFragment#create and can also get rid of unnecessary intermediate PathFragment allocations.

RELNOTES: None

PiperOrigin-RevId: 152145768

src/main/java/com/google/devtools/build/lib/vfs/SearchPath.java

diff --git a/src/main/java/com/google/devtools/build/lib/vfs/SearchPath.java b/src/main/java/com/google/devtools/build/lib/vfs/SearchPath.java
index 7f7d8c7..764ed0d 100644
--- a/src/main/java/com/google/devtools/build/lib/vfs/SearchPath.java
+++ b/src/main/java/com/google/devtools/build/lib/vfs/SearchPath.java
@@ -37,7 +37,7 @@
       return paths;
     }
     for (String p : SEPARATOR.split(searchPath)) {
-      PathFragment pf = new PathFragment(p);
+      PathFragment pf = PathFragment.create(p);
 
       if (pf.isAbsolute()) {
         paths.add(fs.getPath(pf));
@@ -53,7 +53,7 @@
    */
   @Nullable
   public static Path which(List<Path> searchPath, String exe) {
-    PathFragment fragment = new PathFragment(exe);
+    PathFragment fragment = PathFragment.create(exe);
     if (fragment.segmentCount() != 1 || fragment.isAbsolute()) {
       return null;
     }