From 5ae438cf71487d5bc8cb5c0d0e637f976734d110 Mon Sep 17 00:00:00 2001 From: Max Cai Date: Fri, 11 Oct 2013 16:41:41 +0100 Subject: Remove all field initializers and let ctor call clear(). The field initializers have basically caused the compiled method to inline the whole clear() method, which means if ProGuard is not used or failed to inline or remove clear(), there are two big chunks of code that do the same thing. So why not just call clear() from the ctor. Change-Id: Ief71e2b03db2e059b3bfa98309649368089ffab0 --- src/google/protobuf/compiler/javanano/javanano_enum_field.cc | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/google/protobuf/compiler/javanano/javanano_enum_field.cc') diff --git a/src/google/protobuf/compiler/javanano/javanano_enum_field.cc b/src/google/protobuf/compiler/javanano/javanano_enum_field.cc index acb8dc2..420d3c2 100644 --- a/src/google/protobuf/compiler/javanano/javanano_enum_field.cc +++ b/src/google/protobuf/compiler/javanano/javanano_enum_field.cc @@ -89,11 +89,11 @@ EnumFieldGenerator::~EnumFieldGenerator() {} void EnumFieldGenerator:: GenerateMembers(io::Printer* printer) const { printer->Print(variables_, - "public $type$ $name$ = $default$;\n"); + "public $type$ $name$;\n"); if (params_.generate_has()) { printer->Print(variables_, - "public boolean has$capitalized_name$ = false;\n"); + "public boolean has$capitalized_name$;\n"); } } @@ -178,7 +178,7 @@ AccessorEnumFieldGenerator::~AccessorEnumFieldGenerator() {} void AccessorEnumFieldGenerator:: GenerateMembers(io::Printer* printer) const { printer->Print(variables_, - "private int $name$_ = $default$;\n" + "private int $name$_;\n" "public int get$capitalized_name$() {\n" " return $name$_;\n" "}\n" @@ -241,7 +241,7 @@ RepeatedEnumFieldGenerator::~RepeatedEnumFieldGenerator() {} void RepeatedEnumFieldGenerator:: GenerateMembers(io::Printer* printer) const { printer->Print(variables_, - "public $type$[] $name$ = $repeated_default$;\n"); + "public $type$[] $name$;\n"); if (descriptor_->options().packed()) { printer->Print(variables_, "private int $name$MemoizedSerializedSize;\n"); -- cgit v1.1