LuchianMihai commented on PR #15861: URL: https://github.com/apache/nuttx/pull/15861#issuecomment-2667954267
Hi! Regarding the use of ```c #if BOARD_NGPIOINT > 0 #endif ``` It's a personal preference, but, I suggest the to use something like ```c #ifdef BOARD_NGPIOINT #endif ``` If one or more gpio is needed, use ```c #define BOARD_NGPIOINT number ``` else do not define BOARD_NGPIOINT at all. Current implementation assumes that BOARD_NGPIOINT should always be defined and numeric value, Which arguably may introduce undefined behavior. But, it's just my suggestion. -- 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
