diff options
| author |  mstaib <mstaib@google.com> | 2017-06-16 20:45:47 +0200 |
|---|---|---|
| committer |  Philipp Wollermann <philwo@google.com> | 2017-06-19 18:23:34 +0200 |
| commit | b1b794ba78d9f1ccfd013fd56c62c737ec14a4d4 (patch) | |
| tree | f8945a9b31b6380113fd4a454f2adc53b179381a /src/main/java/com/google/devtools/build/lib/skyframe/ConfigurationFragmentFunction.java | |
| parent | d8c286d32513d5696b2ec8c4e7d6ffd4bb332227 (diff) | |
Static-ize ConfigurationBuilderEnvironment.
This inner class has no reason to be non-static that I can see.
This should have no effect on memory if I understand how Java
memory usage works, but it confused me, so I cleaned it up.
RELNOTES: None.
PiperOrigin-RevId: 159255460
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/skyframe/ConfigurationFragmentFunction.java')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/skyframe/ConfigurationFragmentFunction.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/ConfigurationFragmentFunction.java b/src/main/java/com/google/devtools/build/lib/skyframe/ConfigurationFragmentFunction.java index db5aae9acc..642aed2824 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/ConfigurationFragmentFunction.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/ConfigurationFragmentFunction.java @@ -97,7 +97,7 @@ public final class ConfigurationFragmentFunction implements SkyFunction { /** * A {@link ConfigurationEnvironment} implementation that can create dependencies on files. */ - private final class ConfigurationBuilderEnvironment implements ConfigurationEnvironment { + private static final class ConfigurationBuilderEnvironment implements ConfigurationEnvironment { private final PackageProviderForConfigurations packageProvider; ConfigurationBuilderEnvironment(PackageProviderForConfigurations packageProvider) { |