oxidecomputer / amd-apcb

AMD Generic Encapsulated Software Architecture Platform Security Processor Configuration Block manipulation library
Mozilla Public License 2.0
13 stars 1 forks source link

Rename make_serde to make_serde_converter for clarity #56

Closed daym closed 2 years ago

daym commented 2 years ago

Or, even better, to impl_struct_serde_conversion.

The reason is that the make_ macros actually make new structs, fns etc.

But this here just connects existing structs to each other, which is not at all what I would get from the name.

Also, what does it do with serde? Should be in the name.