On Mon, 2025-11-17 at 18:10 -0500, Joel Fernandes wrote:
> // First define trait
> pub(crate) trait FalconUCodeDescriptor {
> fn imem_load_size(&self) -> u32;
> fn dmem_load_size(&self) -> u32;
> fn engine_id_mask(&self) -> u16; // V3-only field, V2 returns 0
> ...
> }Isn't it more idiomatic for engine_id_mask() (and any other field that exists only on one version) to return an Option<u16>?
