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

Reply via email to