Default to packed repeated primitive fields in proto3.

Fixes stepancheg#704. In proto3, repeated primitive fields should be packed by default: https://protobuf.dev/programming-guides/encoding/#packed
rcorre · Jan 17, 2024 · 43df676 · 43df676
1 parent 0d2dbd8
commit 43df676
Showing 1 changed file with 30 additions and 1 deletion.
diff --git a/protobuf-codegen/src/gen/field/mod.rs b/protobuf-codegen/src/gen/field/mod.rs
@@ -188,7 +188,36 @@ impl<'a> FieldGen<'a> {
 
                 FieldKind::Repeated(RepeatedField {
                     elem,
-                    packed: field.field.proto().options.get_or_default().packed(),
+                    packed: field
+                        .field
+                        .proto()
+                        .options
+                        .get_or_default()
+                        .packed
+                        .unwrap_or(match field.message.scope.file_scope.syntax() {
+                            Syntax::Proto2 => false,
+                            // in proto3, repeated primitive types are packed by default
+                            Syntax::Proto3 => match field.field.proto().type_() {
+                                Type::TYPE_DOUBLE
+                                | Type::TYPE_FLOAT
+                                | Type::TYPE_INT64
+                                | Type::TYPE_UINT64
+                                | Type::TYPE_INT32
+                                | Type::TYPE_FIXED64
+                                | Type::TYPE_FIXED32
+                                | Type::TYPE_BOOL
+                                | Type::TYPE_UINT32
+                                | Type::TYPE_SFIXED32
+                                | Type::TYPE_SFIXED64
+                                | Type::TYPE_SINT32
+                                | Type::TYPE_SINT64 => true,
+                                Type::TYPE_STRING
+                                | Type::TYPE_GROUP
+                                | Type::TYPE_MESSAGE
+                                | Type::TYPE_BYTES
+                                | Type::TYPE_ENUM => false,
+                            },
+                        }),
                 })
             }
             RuntimeFieldType::Singular(..) => {