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 16, 2024 · 98b15a7 · 98b15a7
1 parent 0d2dbd8
commit 98b15a7
Showing 1 changed file with 10 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,16 @@ 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,
+                            Syntax::Proto3 => true,
+                        }),
                 })
             }
             RuntimeFieldType::Singular(..) => {