Field-level #[builder(default = __)] easier to use wrong than right about rust-derive-builder HOT 5 CLOSED

Hm, thanks.

There was a discussion about this here (comparison with serde / question 1) and in this comment below:

I feel like the semantics of #[serde(default = "some::Function::foo")] would be too restrictive for derive_builder.

• I suspect a lot of simple cases where delegation to some::Function::foo would be overkill and default="42" is really all you want
• For complex cases it would seem beneficial to delegate to methods on FooBuilder which can inspect all builder fields.

It would have been nice to share the semantics with a popular crate like serde. But this would have significant drawbacks IMO.

The outcome of that discussion was, please delegate to methods if that makes sense (complex stuff), or use arbitrary expressions if you know what you are doing (only really recommended for simple const expressions). The docs clearly recommends delegating to helper methods.

I still believe that 100% serde-behaviour is not a good fit for derive_builder. E.g. in serde you cannot write default="42", instead you have to wrap const-values in a static function. - at least the last time I checked the documentation.

from rust-derive-builder.

I still believe that 100% serde-behaviour is not a good fit for derive_builder. E.g. in serde you cannot write default="42", instead you have to wrap const-values in a static function. - at least the last time I checked the documentation.

This is accurate. Is there maybe a way to check at compile time that the user has either passed:

1. A value literal that implements Copy or
2. A path to a function?

That could disallow the exotic misuses while still allowing the desired simple case.

from rust-derive-builder.

A custom derive can only inspect the source code of the item it is applied to. Nothing more - so we don't know what's in scope and what implements what. We're completely blind to the world. :-)

from rust-derive-builder.

:( bummer. Okay, in that case I'm out of ideas on this one.

from rust-derive-builder.

I thinks it's the best we can do right now. I'll close in the meantime.

from rust-derive-builder.