Rockbox

Tasklist

FS#7457 - Hiding dircache behind a posix API

Attached to Project: Rockbox
Opened by Kévin Ferrare (TiMiD) - Friday, 20 July 2007, 07:48 GMT
Last edited by Peter D'Hoye (petur) - Friday, 20 July 2007, 22:34 GMT
Task Type Patches
Category Operating System/Drivers
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Hi,

I made this patch with which presents dircache (and normal dir api for targets without it) as a standard posix api.

This avoids confusion in app code, so that devs won't mistake and use the dircache-disabled api when dircache is enabled, saving some disks spins
This task depends upon

Closed by  Peter D'Hoye (petur)
Friday, 20 July 2007, 22:34 GMT
Reason for closing:  Accepted
Comment by Kévin Ferrare (TiMiD) - Friday, 20 July 2007, 09:48 GMT
forgot to add the dir.h file.
Patch and add that file to firmware/include
   dir.h (1.5 KiB)
Comment by Kévin Ferrare (TiMiD) - Friday, 20 July 2007, 16:15 GMT
Apply the patch, ignore the "File firmware/common/dir.c is not empty after patch, as expected" error and remove firmware/common/dir.c

works here
Comment by Michael Sevakis (MikeS) - Friday, 20 July 2007, 19:04 GMT
I have never looked into the details of dircache and frankly I think it should work through the normal file APIs without any awareness by the caller of those APIs that it's even there or not. Is that the intent here? It's the impression I get from a quick scan of the patch.

Loading...