[PATCH] KEYS: encrypted: Use designated initializers for match_table_t structs
James Bottomley
James.Bottomley at HansenPartnership.com
Thu Oct 9 12:44:35 UTC 2025
On Thu, 2025-10-09 at 13:58 +0200, Thorsten Blum wrote:
> Use designated initializers for 'key_format_tokens' and 'key_tokens'
> to allow struct fields to be reordered more easily
How does it improve that? The key,value pairs are surrounded by braces
so we just cut and paste the lot anyway.
> and to improve readability.
I don't think I agree with this when looking through the code,
especially because this is the way it's done for *every* option in the
entire key subsystem. So firstly I really don't think it's helpful for
only encrypted keys to be different from everything else and secondly
when I read the code (as I often do to figure out what the options
mean), the additional .token and .pattern just get in the way of what
I'm looking for.
Regards,
James
More information about the Linux-security-module-archive
mailing list