diff options
author | tomlu <tomlu@google.com> | 2018-01-19 10:25:19 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-01-19 10:27:24 -0800 |
commit | e7552c53a5dd65ae496562fc29adecd89e9db5fb (patch) | |
tree | f33fb86896b10777b5652812d161454a2cefbe6c /src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java | |
parent | 26866939e115798cf91d7a02de5417ad81314384 (diff) |
Fix usages of PathFragment segments that will become inefficient.
An upcoming replacement to PathFragment will not have efficient segment semantics, causing code to become unnecessarily inefficient.
RELNOTES: None
PiperOrigin-RevId: 182553098
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java b/src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java index 71449aaf31..3363dc33da 100644 --- a/src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java +++ b/src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java @@ -61,7 +61,8 @@ public final class ArtifactRoot implements Comparable<ArtifactRoot>, Serializabl return asSourceRoot(packageRoot); } else { Path actualRootPath = packageRoot.asPath(); - for (int i = 0; i < repository.getSourceRoot().segmentCount(); i++) { + int segmentCount = repository.getSourceRoot().segmentCount(); + for (int i = 0; i < segmentCount; i++) { actualRootPath = actualRootPath.getParentDirectory(); } return asSourceRoot(Root.fromPath(actualRootPath)); |