diff options
author | 2016-01-14 17:53:54 +0000 | |
---|---|---|
committer | 2016-01-15 09:22:52 +0000 | |
commit | 913d4585d35ffd9c7733d22a8601a8cd44b5d6d5 (patch) | |
tree | 9faad5f98be89e344a3b4ebae254ac9b976de2fc /src/main | |
parent | 5bd8209268c8d5f68dfdd410d59aab389de89bc2 (diff) |
Optimize PackageIdentifier hash code implementation, which has shown up in some recent profiling.
--
MOS_MIGRATED_REVID=112161558
Diffstat (limited to 'src/main')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/cmdline/PackageIdentifier.java | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/cmdline/PackageIdentifier.java b/src/main/java/com/google/devtools/build/lib/cmdline/PackageIdentifier.java index 13f795e160..b29e820b9c 100644 --- a/src/main/java/com/google/devtools/build/lib/cmdline/PackageIdentifier.java +++ b/src/main/java/com/google/devtools/build/lib/cmdline/PackageIdentifier.java @@ -33,7 +33,6 @@ import java.io.ObjectInputStream; import java.io.ObjectOutputStream; import java.io.ObjectStreamException; import java.io.Serializable; -import java.util.Objects; import java.util.concurrent.ExecutionException; import java.util.regex.Pattern; @@ -287,10 +286,9 @@ public final class PackageIdentifier implements Comparable<PackageIdentifier>, S private final PathFragment pkgName; private PackageIdentifier(RepositoryName repository, PathFragment pkgName) { - Preconditions.checkNotNull(repository); - Preconditions.checkNotNull(pkgName); - this.repository = repository; - this.pkgName = Canonicalizer.fragments().intern(pkgName.normalize()); + this.repository = Preconditions.checkNotNull(repository); + this.pkgName = Canonicalizer.fragments().intern( + Preconditions.checkNotNull(pkgName).normalize()); } public static PackageIdentifier parse(String input) throws LabelSyntaxException { @@ -365,7 +363,7 @@ public final class PackageIdentifier implements Comparable<PackageIdentifier>, S @Override public int hashCode() { - return Objects.hash(repository, pkgName); + return 31 * repository.hashCode() + pkgName.hashCode(); } @Override |