xiaoxiang781216 commented on PR #2918:
URL: https://github.com/apache/nuttx-apps/pull/2918#issuecomment-2568603108

   > Thank you @anchao :-) My questions:
   > 
   > 1. Where `LINE_MAX` should be used and where `CONFIG_NSH_LINELEN`?
   
   Yes, the patch plan to replace all XXX_LINELEN to LINE_MAX.
   
   > 2. Where is `LINE_MAX` defined?
   
   here: https://github.com/apache/nuttx/pull/15344
   LINE_MAX is POSIX defined macro.
   
   > 3. Wouldn't that create inconsistency?
   
   no, it improves the consistency, since all places will use the standard 
macro defined by POSIX.
   
   > 4. This seems to require a documentation update? Or at least mention on 
the change and what/where is preferred in regard to above two questions?
   
   


-- 
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