diff options
author | Janak Ramakrishnan <janakr@google.com> | 2015-11-12 23:16:17 +0000 |
---|---|---|
committer | Lukacs Berki <lberki@google.com> | 2015-11-13 10:22:41 +0000 |
commit | 659f80ee8193f606a1591ef31b730f7a60934b70 (patch) | |
tree | 98f5db9965dea5aee738f56ac29ccd9fea8a4b9b /src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java | |
parent | 0363c7d3bb51d30952c39666ca7c9a83587b2983 (diff) |
When getting package roots for exec paths, don't start with the file, which is guaranteed not to be a package. Instead, start with its parent. This will be faster and take less memory.
--
MOS_MIGRATED_REVID=107725767
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java b/src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java index aaf5fca146..a067ba152c 100644 --- a/src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java +++ b/src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java @@ -235,7 +235,7 @@ public class ArtifactFactoryTest { } @Override - public Map<PathFragment, Root> findPackageRoots(Iterable<PathFragment> execPaths) { + public Map<PathFragment, Root> findPackageRootsForFiles(Iterable<PathFragment> execPaths) { Map<PathFragment, Root> result = new HashMap<>(); for (PathFragment execPath : execPaths) { for (PathFragment dir = execPath.getParentDirectory(); dir != null; |