Add VFS library #840
Add VFS library #840
Conversation
bythos14
force-pushed
the
vfs
branch
4 times, most recently
from
a57211d to
73dc67f
Compare
Add header for Iofilemgr VFS functionality. Enables the implementing of custom file system drivers. Co-Authored-By: Princess-of-Sleeping <[email protected]>
include/psp2kern/vfs.h
Outdated
| * VFS Info defining a VFS Implementation | ||
| */ | ||
| typedef struct SceVfsInfo { | ||
| SceVfsOpTable *vfsOps; //!< VFS Implementation operations for mountpoint actions. |
There was a problem hiding this comment.
can you change these member names to snake_case?
There was a problem hiding this comment.
Sure I can do it, as I understand it's for consistency reasons in the SDK, right? The current camelCase is derived from debug strings referencing the fields.
There was a problem hiding this comment.
yeah. i know. couple years ago, princess was also explained that.
it just sdk naming reason :)
include/psp2kern/vfs.h
Outdated
| SceVfsMount *mnt; | ||
| const char *dev; | ||
| unsigned int cmd; | ||
| const void *arg; |
There was a problem hiding this comment.
mixed indent type
i don't remember we use hardtab or softtab in future files.
@Princess-of-Sleeping do you remember this?
There was a problem hiding this comment.
I'll use hardtabs ('\t' characters) since sdstor.h, the most recently added header, does the same.
include/psp2kern/vfs.h
Outdated
| * directory which has been looked-up or accessed. | ||
| */ | ||
| typedef struct SceVfsVnode { | ||
| struct { // vdlock? |
There was a problem hiding this comment.
can we give a name this and below structure?
Add header for Iofilemgr VFS functionality. Enables the implementing of custom file system drivers.
Argument structures derived from vita-header-internal. Credit @Princess-of-Sleeping