Update MessageSet decoding to only take the first values.

Looks like more things have been following upb on behaviors and there's more specific enforcement happening for other languages, so that that for Swift also. If a sub field for proper MessageSet encoding is repeated, only honor the first value for each of the fields, skipping all others. This aligns with the other languages. Add new tests to cover this behavior.
apple · Feb 11, 2025 · eb17584 · eb17584
1 parent 683e459
commit eb17584
Show file tree

Hide file tree

Showing 17 changed files with 10,022 additions and 9,495 deletions.
diff --git a/Protos/SwiftProtobufTests/generated_swift_names_enum_cases.proto b/Protos/SwiftProtobufTests/generated_swift_names_enum_cases.proto
diff --git a/Protos/SwiftProtobufTests/generated_swift_names_enums.proto b/Protos/SwiftProtobufTests/generated_swift_names_enums.proto
@@ -435,6 +435,7 @@ message GeneratedSwiftReservedEnums {
   enum Google_Protobuf_UninterpretedOption { NONE_Google_Protobuf_UninterpretedOption = 0; }
   enum Google_Protobuf_Value { NONE_Google_Protobuf_Value = 0; }
   enum goPackage { NONE_goPackage = 0; }
+  enum gotData { NONE_gotData = 0; }
   enum group { NONE_group = 0; }
   enum groupFieldNumberStack { NONE_groupFieldNumberStack = 0; }
   enum groupSize { NONE_groupSize = 0; }