Hi Andrew,
On Tue, 18 Nov 2025 10:45:43 -0600, Andrew Jones wrote:
I would have written it like below to avoid four redundant lines
case 'm': case 'z': tests[opt == 'm' ? TEST_NO_ZICBOM : TEST_NO_ZICBOZ].enabled = true; tests[TEST_NO_CBO_INVAL].enabled = true; rc = sigaction(SIGILL, &act, NULL); assert(rc == 0); break;
I'll apply this style pattern in future patches, or in a v6 if we end up needing one.
Thanks, Zihong