From d92688eec14040d9a381291bbb881a4b26a4d55d Mon Sep 17 00:00:00 2001 From: cpeyser Date: Tue, 22 May 2018 11:43:48 -0700 Subject: Serialize NestedSetStore$NestedSetSize directly as an ordinal instead of through SerializationContext, to save a byte. PiperOrigin-RevId: 197597779 --- .../build/lib/collect/nestedset/NestedSetCodecWithStore.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/main/java/com/google/devtools/build/lib/collect/nestedset/NestedSetCodecWithStore.java') diff --git a/src/main/java/com/google/devtools/build/lib/collect/nestedset/NestedSetCodecWithStore.java b/src/main/java/com/google/devtools/build/lib/collect/nestedset/NestedSetCodecWithStore.java index 030b0a3332..a673920cd4 100644 --- a/src/main/java/com/google/devtools/build/lib/collect/nestedset/NestedSetCodecWithStore.java +++ b/src/main/java/com/google/devtools/build/lib/collect/nestedset/NestedSetCodecWithStore.java @@ -64,13 +64,13 @@ public class NestedSetCodecWithStore implements ObjectCodec> { if (obj.isEmpty()) { // If the NestedSet is empty, it needs to be assigned to the EMPTY_CHILDREN constant on // deserialization. - context.serialize(NestedSetSize.EMPTY, codedOut); + codedOut.writeEnumNoTag(NestedSetSize.EMPTY.ordinal()); } else if (obj.isSingleton()) { // If the NestedSet is a singleton, we serialize directly as an optimization. - context.serialize(NestedSetSize.SINGLETON, codedOut); + codedOut.writeEnumNoTag(NestedSetSize.SINGLETON.ordinal()); context.serialize(obj.rawChildren(), codedOut); } else { - context.serialize(NestedSetSize.GROUP, codedOut); + codedOut.writeEnumNoTag(NestedSetSize.GROUP.ordinal()); FingerprintComputationResult fingerprintComputationResult = nestedSetStore.computeFingerprintAndStore((Object[]) obj.rawChildren(), context); context.addFutureToBlockWritingOn(fingerprintComputationResult.writeStatus()); @@ -87,7 +87,7 @@ public class NestedSetCodecWithStore implements ObjectCodec> { } Order order = context.deserialize(codedIn); - NestedSetSize nestedSetSize = context.deserialize(codedIn); + NestedSetSize nestedSetSize = NestedSetSize.values()[codedIn.readEnum()]; switch (nestedSetSize) { case EMPTY: return NestedSetBuilder.emptySet(order); -- cgit v1.2.3