Hoist "continue" statement outside of inner loop.

RELNOTES: None
PiperOrigin-RevId: 182546239

1 files changed, 4 insertions, 4 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/buildtool/SymlinkForest.java b/src/main/java/com/google/devtools/build/lib/buildtool/SymlinkForest.java
index 4b4f7cc5c4..13b57cf467 100644
--- a/src/main/java/com/google/devtools/build/lib/buildtool/SymlinkForest.java
+++ b/src/main/java/com/google/devtools/build/lib/buildtool/SymlinkForest.java
@@ -118,12 +118,12 @@ class SymlinkForest {
     Map<PackageIdentifier, Set<Root>> dirRootsMap = Maps.newTreeMap();
     for (Map.Entry<PackageIdentifier, Root> entry : packageRoots.entrySet()) {
       PackageIdentifier pkgId = entry.getKey();
+      if (pkgId.equals(Label.EXTERNAL_PACKAGE_IDENTIFIER)) {
+        // This isn't a "real" package, don't add it to the symlink tree.
+        continue;
+      }
       Root pkgRoot = entry.getValue();
       for (int i = 1; i <= pkgId.getPackageFragment().segmentCount(); i++) {
-        if (pkgId.equals(Label.EXTERNAL_PACKAGE_IDENTIFIER)) {
-          // This isn't a "real" package, don't add it to the symlink tree.
-          continue;
-        }
         PackageIdentifier dir = createInRepo(pkgId, pkgId.getPackageFragment().subFragment(0, i));
         Set<Root> roots = dirRootsMap.computeIfAbsent(dir, k -> Sets.newHashSet());
         roots.add(pkgRoot);