gpoulios commented on code in PR #16309: URL: https://github.com/apache/nuttx/pull/16309#discussion_r2074805787
########## include/nuttx/tee.h: ########## @@ -198,6 +198,10 @@ struct tee_ioctl_shm_register_fd_data #define TEE_IOCTL_LOGIN_REE_KERNEL 0x80000000 +/* Macro to help calculate the total size of n params */ + +#define TEE_IOCTL_PARAM_SIZE(x) (sizeof(struct tee_ioctl_param) * (x)) Review Comment: It's actually 76 (up to column 77), and if I left-align `(sizeof(...` it goes over this vertical limit. I could start breaking it using `\` but it would still require 3 lines, unless I do it like this: ``` #define TEE_IOCTL_PARAM_SIZE(x) \ (sizeof(struct tee_ioctl_param) * (x)) ``` ...which is still not aligned. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org