You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/07/01 18:40:59 UTC

[GitHub] [incubator-nuttx] davids5 commented on a change in pull request #3953: Minor fix for local time process

davids5 commented on a change in pull request #3953:
URL: https://github.com/apache/incubator-nuttx/pull/3953#discussion_r662515753



##########
File path: libs/libc/time/lib_localtime.c
##########
@@ -363,9 +365,9 @@ FAR char *tzname[2] =
 
 static int_fast32_t detzcode(FAR const char *codep);
 static int_fast64_t detzcode64(FAR const char *codep);
-static int  differ_by_repeat(time_t t1, time_t t0);
+static int differ_by_repeat(time_t t1, time_t t0);
 static FAR const char *getzname(FAR const char *strp);
-static FAR const char *getqzname(FAR const char *strp, const int delim);
+static FAR const char *getqzname(FAR const char *strp, int delim);

Review comment:
       @xiaoxiang781216 Why are you dropping ALL the const specifiers?




-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org