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 2023/01/04 08:26:02 UTC

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8027: net/local: rename NET_LOCAL_VFS_PATH to follow linux

pkarashchenko commented on code in PR #8027:
URL: https://github.com/apache/nuttx/pull/8027#discussion_r1061234267


##########
net/local/local_fifo.c:
##########
@@ -66,6 +66,11 @@
 static void local_format_name(FAR const char *inpath, FAR char *outpath,
                               FAR const char *suffix, int32_t id)
 {
+  if (strcmp(inpath, CONFIG_NET_LOCAL_VFS_PATH) == 0)
+    {
+      inpath += strlen(CONFIG_NET_LOCAL_VFS_PATH);

Review Comment:
   Could you please provide some example on what will be the `inpath` before and after this operation?
   I mean that it seems to be something like `/var/run/mypath` converted to `/mypath`?



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