Merge pull request #19 from YuhanLiin/reduce-generated-code

Reduce generated code

Author: YuhanLiin

CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## Unreleased
+
+### Added
+
+- Add `no_accessors` configuration to reduce generated file size
+- Add new setting to disable generating `MAX_SIZE` calculations
+
+### Changed
+
+- Use derives for `PartialEq` and `Default` when possible
+
 ## 0.2.0
 
 ### Added
@@ -11,8 +22,8 @@
 - Add `field_lifetime` configuration to set lifetime of message fields
 - Add container trait impls for `Cow`
 - Add const constructor `_new` to hazzer structs and add const to all hazzer methods
-- Add `Generator::configure_many`
-- Add `Config::recursive_field`
+- Add `Generator::configure_many` method for configuring multiple paths at once
+- Add `recursive_field` configuration to box and handle max size for recursive fields
 
 ### Changed
 

examples/file-descriptor-set/build.rs

@@ -6,7 +6,23 @@ fn main() {
         .encode_decode(EncodeDecode::DecodeOnly)
         .configure(
             ".",
-            Config::new().no_clone_impl(true).no_partial_eq_impl(true),
+            Config::new()
+                .no_clone_impl(true)
+                .no_partial_eq_impl(true)
+                .no_accessors(true),
+        )
+        // Override minimal accessors setting for specific paths, since the generator calls `set_`
+        // APIs on specific messages
+        .configure_many(
+            &[
+                ".google.protobuf.DescriptorProto",
+                ".google.protobuf.FieldDescriptorProto",
+                ".google.protobuf.FieldOptions",
+                ".google.protobuf.OneofDescriptorProto",
+                ".google.protobuf.MessageOptions",
+                ".google.protobuf.EnumValueDescriptorProto",
+            ],
+            Config::new().no_accessors(false),
         )
         .compile_protos(&["google/protobuf/descriptor.proto"], "descriptor.rs")
         .unwrap();

micropb-gen/src/config.rs

@@ -423,6 +423,13 @@ config_decl! {
     /// ```
     field_lifetime: [deref] Option<String>,
 
+    /// Disable field accessors.
+    ///
+    /// Do not generate accessors for this field other than the getter method on optional fields,
+    /// which is required by the encoding logic. This won't reduce the compiled code size, but it
+    /// will significantly reduce the size of the output source file.
+    no_accessors: Option<bool>,
+
     // Type configs
 
     /// Override the integer size of Protobuf enums.