xiaoxiang781216 commented on code in PR #7317: URL: https://github.com/apache/incubator-nuttx/pull/7317#discussion_r996337161
########## fs/vfs/fs_rename.c: ########## @@ -90,7 +90,7 @@ static int pseudorename(FAR const char *oldpath, FAR struct inode *oldinode, { inode_release(newinode); ret = OK; - goto errout; /* Bad naming, this is not an error case. */ + goto errout; /* Same name, this is not an error case. */ Review Comment: @acassis the message is reported by mv command, not function Here is my test on WSL2: ``` $ cat test.c #include <stdio.h> int main(int argc, char *argv[]) { return rename(argv[1], argv[2]); } $ gcc test.c $ ./a.out a.out a.out $ echo $? 0 ``` -- 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