-
Notifications
You must be signed in to change notification settings - Fork 31
io: use fios2 #90
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: vita
Are you sure you want to change the base?
io: use fios2 #90
Conversation
| #define SCE_FIOS_OPATTR_INITIALIZER { 0, 0, 0, 0, 0, 0, 0, 0 } | ||
| #define SCE_FIOS_RAM_CACHE_CONTEXT_INITIALIZER { sizeof(SceFiosRamCacheContext), 0, (64 * 1024), NULL, NULL, 0, {0, 0, 0} } | ||
|
|
||
| typedef enum SceFiosWhence { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those should go in headers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know
| typedef struct SceFiosRamCacheContext { | ||
| size_t sizeOfContext; | ||
| size_t workBufferSize; | ||
| size_t blockSize; | ||
| void *pWorkBuffer; | ||
| const char *pPath; | ||
| intptr_t flags; | ||
| intptr_t reserved[3]; | ||
| } SceFiosRamCacheContext; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we use snake_case instead of camelCase?
eg;
pPtr->ptrpCallback->callbackpCallbackContext->callback_contextuserTag->user_tagor justtaguserPtr->user_ptror justuserpReserved->reservedsizeOfContext->size_of_contextworkBufferSize->work_buffer_size,work_buf_sizeorbuf_sizeblockSize->block_sizepWorkBuffer->work_buffer,bufferor justbuf- ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no problem. I'm still reversing some of the stuff and these are leftovers from vitashell.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TheOfficialFloW can you allow changing the license to LGPL or others?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
|
@frangarcj you might want to take a look at #92 because i'm pretty sure fios rename behaves the same |
Missing: