mirrors/arti.git/crates/tor-config/src/derive.rs, branch arti-v2.4.0

Apply 3 suggestion(s) to 2 file(s)

2026-05-27T12:05:52Z

Co-authored-by: gabi-250

config: Add a dd(TorConfig) attribute to override apply_defaults().

2026-05-27T12:05:52Z

config: Add a method to fill in a builder with unset defaults

2026-05-27T12:05:52Z

This new method modifies a builder by replacing any unset values that have a default with that default. We're using this method so that we can re-serialize a builder into a `ConfigurationTree` with all of its default values included. In all cases, `b.apply_defaults()?; b.build()` should produce the same output as `b.build()`. The interesting parts of this commit are in tor_config::load and tor_config::derive. The rest of this commit just adds `apply_defaults` to other builders that _aren't_ made with `derive_deftly(TorConfig)`.

dd(TorConfig): change the order of default vs magic

2026-05-27T12:05:51Z

This affects the automatic builder code made by our derive_deftly macro. It is only relevant (for now) in the case of the `NonZero<>` types and their special handling. Previously, when a builder contained Option, and we wanted to generate a configuration holding T, we would _first_ apply a transformation from Option to Option and _second_ unwrap the result or apply a default. Now, we _first_ convert from Option to U by applying a default, and only _then_ perform any necessary conversion from U and T. This is only relevant in the case where U and T are different. It simplifies writing the defaults for `NonZero` options, and will significantly simplify the logic for setting builder defaults.

tor-config::derive: use cfg(true) and cfg(false)

2026-05-07T12:08:01Z

Rust 1.88 added these, so we no longer have to use `any()` for false and `all()` for true.

Fix word duplicate typos

2026-03-15T20:18:56Z

tor-config: future-proof #[cfg] attribute.

2026-02-24T13:05:39Z

tor-config: Add an option to reject configured-out options

2026-02-24T13:05:39Z

tor-config: Have macros document more types and fns

2026-02-24T13:05:39Z

tor-config: Fix cfg() on sub_builders

2026-02-24T13:05:39Z

Previously, it used the sub_builder rule, which doesn't make sense when the value is a stub place-holder to tell us whether to warn.