[GitHub] [incubator-nuttx] Donny9 commented on a diff in pull request #6803: fs/directory: Using file api to implement operations on directories

[GitBox <gi...@apache.org>]

Donny9 commented on code in PR #6803:
URL: https://github.com/apache/incubator-nuttx/pull/6803#discussion_r939681648


##########
arch/arm/src/cxd56xx/cxd56_powermgr_procfs.c:
##########
@@ -787,9 +777,9 @@ static int cxd56_powermgr_procfs_rewinddir(FAR struct fs_dirent_s *dir)
 {
   struct cxd56_powermgr_procfs_dir_s *procfs;
 
-  DEBUGASSERT(dir && dir->u.procfs);
+  DEBUGASSERT(dir);
 
-  procfs = (struct cxd56_powermgr_procfs_dir_s *)dir->u.procfs;
+  procfs = (FAR struct cxd56_powermgr_procfs_dir_s *)dir;

Review Comment:
   Done!



##########
arch/arm/src/cxd56xx/cxd56_powermgr_procfs.c:
##########
@@ -711,7 +710,7 @@ static int cxd56_powermgr_procfs_opendir(const char *relpath,
     }
 
   procfs->index = 0;
-  dir->u.procfs = (void *)procfs;
+  *dir = (FAR struct fs_dirent_s *)procfs;

Review Comment:
   Done



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