fjpanag commented on code in PR #1328:
URL: 
https://github.com/apache/incubator-nuttx-apps/pull/1328#discussion_r983581092


##########
netutils/thttpd/tdate_parse.c:
##########
@@ -93,11 +93,14 @@ static int strlong_compare(const void *v1, const void *v2)
 #ifdef HAVE_DAY_OF_WEEK /* Day of week not yet supported by NuttX */
 static int strlong_search(char *str, struct strlong *tab, int n, long *lP)

Review Comment:
   I didn't go through the code, so I don't know what `lP` means or what would 
be a more appropriate name.
   Do you have a 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

Reply via email to