commit 14db507c284a936dc14559f095a4629d5c52f5d8
author Ulf Adams <ulfjack@google.com> Mon Mar 20 16:00:51 2017 +0000
committer Yue Gan <yueg@google.com> Tue Mar 21 12:48:48 2017 +0000
tree 16529521638a0984463db7cb53071a40671eab75
parent 41ac1e041484205f1558f30fb5ff75e15acbe39e

Add ActionInput.getExecPath, which returns a PathFragment

Update some callers to use getExecPath, which generally results in less
intermediate garbage generation (almost all callers need a PathFragment, not
a String).

Another small step towards #1593.

--
PiperOrigin-RevId: 150631279
MOS_MIGRATED_REVID=150631279

src/main/java/com/google/devtools/build/lib/actions/ActionInputHelper.java

diff --git a/src/main/java/com/google/devtools/build/lib/actions/ActionInputHelper.java b/src/main/java/com/google/devtools/build/lib/actions/ActionInputHelper.java
index 4ca4a1b..c63fd5b 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/ActionInputHelper.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/ActionInputHelper.java
@@ -69,6 +69,7 @@
    */
   private static class BasicActionInput implements ActionInput {
     private final String path;
+
     public BasicActionInput(String path) {
       this.path = Preconditions.checkNotNull(path);
     }
@@ -79,6 +80,11 @@
     }
 
     @Override
+    public PathFragment getExecPath() {
+      return new PathFragment(path);
+    }
+
+    @Override
     public int hashCode() {
       return path.hashCode();
     }