Revert "Merge pull request #10470 from jhump/jh/source-code-info-pseudo-options" (#10658)

This reverts commit dd052c9dc6, reversing
changes made to 0d3eaed6eb.

src/google/protobuf/compiler/parser.cc

@@ -430,11 +430,6 @@ Parser::LocationRecorder::LocationRecorder(const LocationRecorder& parent) {
   Init(parent, parent.source_code_info_);
 }
 
-Parser::LocationRecorder::LocationRecorder(const LocationRecorder& parent,
-                                           SourceCodeInfo* source_code_info) {
-  Init(parent, source_code_info);
-}
-
 Parser::LocationRecorder::LocationRecorder(const LocationRecorder& parent,
                                            int path1,
                                            SourceCodeInfo* source_code_info) {
@@ -1264,22 +1259,13 @@ bool Parser::ParseDefaultAssignment(
     field->clear_default_value();
   }
 
-  LocationRecorder location(field_location,
-                            FieldDescriptorProto::kDefaultValueFieldNumber);
-
   DO(Consume("default"));
   DO(Consume("="));
 
-  // We don't need to create separate spans in source code info for name and value,
+  LocationRecorder location(field_location,
-  // since there's no way to represent them distinctly in a location path. But we will
+                            FieldDescriptorProto::kDefaultValueFieldNumber);
-  // want a separate recorder for the value, just to have more precise location info
+  location.RecordLegacyLocation(field,
-  // in error messages. So we let it create a location in no_op, so it doesn't add a
+                                DescriptorPool::ErrorCollector::DEFAULT_VALUE);
-  // span to the file descriptor.
-  SourceCodeInfo no_op;
-  LocationRecorder value_location(location, &no_op);
-  value_location.RecordLegacyLocation(
-      field, DescriptorPool::ErrorCollector::DEFAULT_VALUE);
-
   std::string* default_value = field->mutable_default_value();
 
   if (!field->has_type()) {
@@ -1413,23 +1399,13 @@ bool Parser::ParseJsonName(FieldDescriptorProto* field,
 
   LocationRecorder location(field_location,
                             FieldDescriptorProto::kJsonNameFieldNumber);
-
+  location.RecordLegacyLocation(field,
-  // We don't need to create separate spans in source code info for name and value,
+                                DescriptorPool::ErrorCollector::OPTION_NAME);
-  // since there's no way to represent them distinctly in a location path. But we will
-  // want a separate recorder for them, just to have more precise location info
-  // in error messages. So we let them create a location in no_op, so they don't
-  // add a span to the file descriptor.
-  SourceCodeInfo no_op;
-  {
-    LocationRecorder name_location(location, &no_op);
-    name_location.RecordLegacyLocation(
-        field, DescriptorPool::ErrorCollector::OPTION_NAME);
 
   DO(Consume("json_name"));
-  }
   DO(Consume("="));
 
-  LocationRecorder value_location(location, &no_op);
+  LocationRecorder value_location(location);
   value_location.RecordLegacyLocation(
       field, DescriptorPool::ErrorCollector::OPTION_VALUE);
 

src/google/protobuf/compiler/parser.h

@@ -245,10 +245,6 @@ class PROTOBUF_EXPORT Parser {
     LocationRecorder(const LocationRecorder& parent, int path1, int path2);
 
     // Creates a recorder that generates locations into given source code info.
-    LocationRecorder(const LocationRecorder& parent,
-                     SourceCodeInfo* source_code_info);
-    // Creates a recorder that generates locations into given source code info
-    // and calls AddPath() one time.
     LocationRecorder(const LocationRecorder& parent, int path1,
                      SourceCodeInfo* source_code_info);
 

src/google/protobuf/compiler/parser_unittest.cc

@@ -3606,19 +3606,18 @@ TEST_F(SourceInfoTest, FieldOptions) {
   EXPECT_TRUE(
       Parse("message Foo {"
             "  optional int32 bar = 1 "
-            "$a$[$b$default=123$c$, $d$opt1=123$e$, "
+            "$a$[default=$b$123$c$,$d$opt1=123$e$,"
-            "$f$opt2='hi'$g$, $h$json_name='barBar'$i$]$j$;"
+            "$f$opt2='hi'$g$]$h$;"
             "}\n"));
 
   const FieldDescriptorProto& field = file_.message_type(0).field(0);
   const UninterpretedOption& option1 = field.options().uninterpreted_option(0);
   const UninterpretedOption& option2 = field.options().uninterpreted_option(1);
 
-  EXPECT_TRUE(HasSpan('a', 'j', field.options()));
+  EXPECT_TRUE(HasSpan('a', 'h', field.options()));
   EXPECT_TRUE(HasSpan('b', 'c', field, "default_value"));
   EXPECT_TRUE(HasSpan('d', 'e', option1));
   EXPECT_TRUE(HasSpan('f', 'g', option2));
-  EXPECT_TRUE(HasSpan('h', 'i', field, "json_name"));
 
   // Ignore these.
   EXPECT_TRUE(HasSpan(file_));