JianyuWang0623 commented on code in PR #2737:
URL: https://github.com/apache/nuttx-apps/pull/2737#discussion_r1807897094


##########
nshlib/nsh_parse.c:
##########
@@ -2466,7 +2487,7 @@ static int nsh_nice(FAR struct nsh_vtbl_s *vtbl, FAR char 
**ppcmd,
 
 #ifdef CONFIG_NSH_CMDPARMS
 static int nsh_parse_cmdparm(FAR struct nsh_vtbl_s *vtbl, FAR char *cmdline,
-                             FAR const char *redirfile_out)
+                             FAR const struct nsh_param_s *exec)

Review Comment:
   done, but shadows warning occured, should we update as below? (as current 
version does)
   ```
   nsh_parse.c:792:26: warning: declaration of ‘param’ shadows a parameter 
[-Wshadow]
     792 |       struct sched_param param;
         |                          ^~~~~
   nsh_parse.c:637:54: note: shadowed declaration is here
     637 |                        FAR const struct nsh_param_s *param)
         |
   ```
   ```diff
   - struct sched_param param;
   + struct sched_param sched;
   ```



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