Drive-by clean-up: Make it slightly more obvious that one of the PackageFunction constructors is only used in tests and annotate the missing variable.
RELNOTES: None.
// limitations under the License.
package com.google.devtools.build.lib.skyframe;
+import com.google.common.annotations.VisibleForTesting;
import com.google.common.base.Throwables;
import com.google.common.cache.Cache;
import com.google.common.collect.ImmutableCollection;
@@ -132,6 +133,7 @@ public class PackageFunction implements SkyFunction {
this.actionOnIOExceptionReadingBuildFile = actionOnIOExceptionReadingBuildFile;
}
+ @VisibleForTesting
public PackageFunction(
PackageFactory packageFactory,
CachingPackageLocator pkgLocator,
@@ -148,7 +150,7 @@ public class PackageFunction implements SkyFunction {
astCache,
numPackagesLoaded,
skylarkImportLookupFunctionForInlining,
- null,
+ /*packageProgress=*/ null,
ActionOnIOExceptionReadingBuildFile.UseOriginalIOException.INSTANCE);
}
