| Index: third_party/protobuf/src/google/protobuf/compiler/java/java_enum_field.cc
|
| diff --git a/third_party/protobuf/src/google/protobuf/compiler/java/java_enum_field.cc b/third_party/protobuf/src/google/protobuf/compiler/java/java_enum_field.cc
|
| index 3e54be3d5a495bfef1bbd1dfdea165d48c992ed9..2e916c5679df22aab9e496284e4316d5d2f2292a 100644
|
| --- a/third_party/protobuf/src/google/protobuf/compiler/java/java_enum_field.cc
|
| +++ b/third_party/protobuf/src/google/protobuf/compiler/java/java_enum_field.cc
|
| @@ -76,6 +76,11 @@ void SetEnumVariables(const FieldDescriptor* descriptor,
|
| (*variables)["deprecation"] = descriptor->options().deprecated()
|
| ? "@java.lang.Deprecated " : "";
|
| (*variables)["on_changed"] = "onChanged();";
|
| + // Use deprecated valueOf() method to be compatible with old generated code
|
| + // for v2.5.0/v2.6.1.
|
| + // TODO(xiaofeng): Use "forNumber" when we no longer support compatibility
|
| + // with v2.5.0/v2.6.1.
|
| + (*variables)["for_number"] = "valueOf";
|
|
|
| if (SupportFieldPresence(descriptor->file())) {
|
| // For singular messages and builders, one bit is used for the hasField bit.
|
| @@ -191,7 +196,7 @@ GenerateMembers(io::Printer* printer) const {
|
| WriteFieldDocComment(printer, descriptor_);
|
| printer->Print(variables_,
|
| "$deprecation$public $type$ get$capitalized_name$() {\n"
|
| - " $type$ result = $type$.forNumber($name$_);\n"
|
| + " $type$ result = $type$.$for_number$($name$_);\n"
|
| " return result == null ? $unknown$ : result;\n"
|
| "}\n");
|
| }
|
| @@ -224,7 +229,7 @@ GenerateBuilderMembers(io::Printer* printer) const {
|
| WriteFieldDocComment(printer, descriptor_);
|
| printer->Print(variables_,
|
| "$deprecation$public $type$ get$capitalized_name$() {\n"
|
| - " $type$ result = $type$.forNumber($name$_);\n"
|
| + " $type$ result = $type$.$for_number$($name$_);\n"
|
| " return result == null ? $unknown$ : result;\n"
|
| "}\n");
|
| WriteFieldDocComment(printer, descriptor_);
|
| @@ -304,7 +309,7 @@ GenerateParsingCode(io::Printer* printer) const {
|
| } else {
|
| printer->Print(variables_,
|
| "int rawValue = input.readEnum();\n"
|
| - "$type$ value = $type$.forNumber(rawValue);\n"
|
| + "$type$ value = $type$.$for_number$(rawValue);\n"
|
| "if (value == null) {\n");
|
| if (PreserveUnknownFields(descriptor_->containing_type())) {
|
| printer->Print(variables_,
|
| @@ -398,7 +403,8 @@ GenerateMembers(io::Printer* printer) const {
|
| printer->Print(variables_,
|
| "$deprecation$public $type$ get$capitalized_name$() {\n"
|
| " if ($has_oneof_case_message$) {\n"
|
| - " $type$ result = $type$.forNumber((java.lang.Integer) $oneof_name$_);\n"
|
| + " $type$ result = $type$.$for_number$(\n"
|
| + " (java.lang.Integer) $oneof_name$_);\n"
|
| " return result == null ? $unknown$ : result;\n"
|
| " }\n"
|
| " return $default$;\n"
|
| @@ -436,7 +442,8 @@ GenerateBuilderMembers(io::Printer* printer) const {
|
| printer->Print(variables_,
|
| "$deprecation$public $type$ get$capitalized_name$() {\n"
|
| " if ($has_oneof_case_message$) {\n"
|
| - " $type$ result = $type$.forNumber((java.lang.Integer) $oneof_name$_);\n"
|
| + " $type$ result = $type$.$for_number$(\n"
|
| + " (java.lang.Integer) $oneof_name$_);\n"
|
| " return result == null ? $unknown$ : result;\n"
|
| " }\n"
|
| " return $default$;\n"
|
| @@ -493,7 +500,7 @@ GenerateParsingCode(io::Printer* printer) const {
|
| } else {
|
| printer->Print(variables_,
|
| "int rawValue = input.readEnum();\n"
|
| - "$type$ value = $type$.forNumber(rawValue);\n"
|
| + "$type$ value = $type$.$for_number$(rawValue);\n"
|
| "if (value == null) {\n");
|
| if (PreserveUnknownFields(descriptor_->containing_type())) {
|
| printer->Print(variables_,
|
| @@ -606,7 +613,7 @@ GenerateMembers(io::Printer* printer) const {
|
| " new com.google.protobuf.Internal.ListAdapter.Converter<\n"
|
| " java.lang.Integer, $type$>() {\n"
|
| " public $type$ convert(java.lang.Integer from) {\n"
|
| - " $type$ result = $type$.forNumber(from);\n"
|
| + " $type$ result = $type$.$for_number$(from);\n"
|
| " return result == null ? $unknown$ : result;\n"
|
| " }\n"
|
| " };\n");
|
| @@ -839,7 +846,7 @@ GenerateParsingCode(io::Printer* printer) const {
|
| } else {
|
| printer->Print(variables_,
|
| "int rawValue = input.readEnum();\n"
|
| - "$type$ value = $type$.forNumber(rawValue);\n"
|
| + "$type$ value = $type$.$for_number$(rawValue);\n"
|
| "if (value == null) {\n");
|
| if (PreserveUnknownFields(descriptor_->containing_type())) {
|
| printer->Print(variables_,
|
| @@ -887,8 +894,8 @@ GenerateSerializationCode(io::Printer* printer) const {
|
| if (descriptor_->is_packed()) {
|
| printer->Print(variables_,
|
| "if (get$capitalized_name$List().size() > 0) {\n"
|
| - " output.writeRawVarint32($tag$);\n"
|
| - " output.writeRawVarint32($name$MemoizedSerializedSize);\n"
|
| + " output.writeUInt32NoTag($tag$);\n"
|
| + " output.writeUInt32NoTag($name$MemoizedSerializedSize);\n"
|
| "}\n"
|
| "for (int i = 0; i < $name$_.size(); i++) {\n"
|
| " output.writeEnumNoTag($name$_.get(i));\n"
|
| @@ -920,7 +927,7 @@ GenerateSerializedSizeCode(io::Printer* printer) const {
|
| "if (!get$capitalized_name$List().isEmpty()) {"
|
| " size += $tag_size$;\n"
|
| " size += com.google.protobuf.CodedOutputStream\n"
|
| - " .computeRawVarint32Size(dataSize);\n"
|
| + " .computeUInt32SizeNoTag(dataSize);\n"
|
| "}");
|
| } else {
|
| printer->Print(variables_,
|
|
|