From 011a325e32c05f6e4817dcdc555615e6b7b6c102 Mon Sep 17 00:00:00 2001 From: Kevin Ferrare Date: Fri, 20 Jul 2007 17:06:55 +0000 Subject: Makes apps and plugins interract with directories using a posix-like api instead of calling dircache / simulator functions (no additionnal layer added, only a cosmetic change) git-svn-id: svn://svn.rockbox.org/rockbox/trunk@13943 a1c6a512-1295-4272-9138-f99709370657 --- apps/plugins/doom/rockdoom.c | 8 +++--- apps/plugins/doom/rockmacros.h | 11 +------- apps/plugins/properties.c | 53 +-------------------------------------- apps/plugins/rockboy/rockboy.c | 6 ++--- apps/plugins/rockboy/rockmacros.h | 13 ++-------- apps/plugins/rockpaint.c | 22 ++++++++-------- 6 files changed, 22 insertions(+), 91 deletions(-) (limited to 'apps/plugins') diff --git a/apps/plugins/doom/rockdoom.c b/apps/plugins/doom/rockdoom.c index 90c446b9eb..a9e348160f 100644 --- a/apps/plugins/doom/rockdoom.c +++ b/apps/plugins/doom/rockdoom.c @@ -329,7 +329,7 @@ int Dbuild_filelistm(struct menu_item **names, char *firstentry, char *directory char *startpt; struct menu_item *temp; - filedir=opendir(directory); + filedir=rb->opendir(directory); if(filedir==NULL) { @@ -345,8 +345,8 @@ int Dbuild_filelistm(struct menu_item **names, char *firstentry, char *directory i++; // Reset the directory - closedir(filedir); - filedir=opendir(directory); + rb->closedir(filedir); + filedir=rb->opendir(directory); i++; temp=malloc(i*sizeof(struct opt_items)); @@ -365,7 +365,7 @@ int Dbuild_filelistm(struct menu_item **names, char *firstentry, char *directory i++; } } - closedir(filedir); + rb->closedir(filedir); *names=temp; return i; } diff --git a/apps/plugins/doom/rockmacros.h b/apps/plugins/doom/rockmacros.h index 86de4cbe13..1541ef48fd 100644 --- a/apps/plugins/doom/rockmacros.h +++ b/apps/plugins/doom/rockmacros.h @@ -40,26 +40,17 @@ char *my_strtok( char * s, const char * delim ); #define read_line(a,b,c) rb->read_line((a),(b),(c)) #ifdef SIMULATOR -#undef opendir -#undef closedir -#undef mkdir #undef open #undef lseek #undef filesize -#define opendir(a) rb->sim_opendir((a)) -#define closedir(a) rb->sim_closedir((a)) -#define mkdir(a) rb->sim_mkdir((a)) #define open(a,b) rb->sim_open((a),(b)) #define lseek(a,b,c) rb->sim_lseek((a),(b),(c)) #define filesize(a) rb->sim_filesize((a)) #else /* !SIMULATOR */ -#define opendir(a) rb->opendir((a)) -#define closedir(a) rb->closedir((a)) -#define filesize(a) rb->filesize((a)) -#define mkdir(a) rb->mkdir((a)) #define open(a,b) my_open((a),(b)) #define close(a) my_close((a)) #define lseek(a,b,c) rb->lseek((a),(b),(c)) +#define filesize(a) rb->filesize((a)) #endif /* !SIMULATOR */ #define strtok(a,b) my_strtok((a),(b)) diff --git a/apps/plugins/properties.c b/apps/plugins/properties.c index 3b4f2797af..86817e2cc7 100644 --- a/apps/plugins/properties.c +++ b/apps/plugins/properties.c @@ -59,30 +59,18 @@ static bool file_properties(char* selected_file) { bool found = false; char tstr[MAX_PATH]; -#ifdef HAVE_DIRCACHE - DIRCACHED* dir; - struct dircache_entry* entry; -#else DIR* dir; struct dirent* entry; -#endif + char* ptr = rb->strrchr(selected_file, '/') + 1; int dirlen = (ptr - selected_file); rb->strncpy(tstr, selected_file, dirlen); tstr[dirlen] = 0; -#ifdef HAVE_DIRCACHE - dir = rb->opendir_cached(tstr); -#else dir = rb->opendir(tstr); -#endif if (dir) { -#ifdef HAVE_DIRCACHE - while(0 != (entry = rb->readdir_cached(dir))) -#else while(0 != (entry = rb->readdir(dir))) -#endif { if(!rb->strcmp(entry->d_name, selected_file+dirlen)) { @@ -103,11 +91,7 @@ static bool file_properties(char* selected_file) break; } } -#ifdef HAVE_DIRCACHE - rb->closedir_cached(dir); -#else rb->closedir(dir); -#endif } return found; } @@ -128,30 +112,17 @@ static bool _dir_properties(DPS* dps) and informs the user of the progress */ bool result; int dirlen; -#ifdef HAVE_DIRCACHE - DIRCACHED* dir; - struct dircache_entry* entry; -#else DIR* dir; struct dirent* entry; -#endif result = true; dirlen = rb->strlen(dps->dirname); -#ifdef HAVE_DIRCACHE - dir = rb->opendir_cached(dps->dirname); -#else dir = rb->opendir(dps->dirname); -#endif if (!dir) return false; /* open error */ /* walk through the directory content */ -#ifdef HAVE_DIRCACHE - while(result && (0 != (entry = rb->readdir_cached(dir)))) -#else while(result && (0 != (entry = rb->readdir(dir)))) -#endif { /* append name to current directory */ rb->snprintf(dps->dirname+dirlen, dps->len-dirlen, "/%s", @@ -189,12 +160,7 @@ static bool _dir_properties(DPS* dps) result = false; rb->yield(); } -#ifdef HAVE_DIRCACHE - rb->closedir_cached(dir); -#else rb->closedir(dir); -#endif - return result; } @@ -256,30 +222,17 @@ enum plugin_status plugin_start(struct plugin_api* api, void* file) /* determine if it's a file or a directory */ bool found = false; -#ifdef HAVE_DIRCACHE - DIRCACHED* dir; - struct dircache_entry* entry; -#else DIR* dir; struct dirent* entry; -#endif char* ptr = rb->strrchr((char*)file, '/') + 1; int dirlen = (ptr - (char*)file); rb->strncpy(str_dirname, (char*)file, dirlen); str_dirname[dirlen] = 0; -#ifdef HAVE_DIRCACHE - dir = rb->opendir_cached(str_dirname); -#else dir = rb->opendir(str_dirname); -#endif if (dir) { -#ifdef HAVE_DIRCACHE - while(0 != (entry = rb->readdir_cached(dir))) -#else while(0 != (entry = rb->readdir(dir))) -#endif { if(!rb->strcmp(entry->d_name, file+dirlen)) { @@ -288,11 +241,7 @@ enum plugin_status plugin_start(struct plugin_api* api, void* file) break; } } -#ifdef HAVE_DIRCACHE - rb->closedir_cached(dir); -#else rb->closedir(dir); -#endif } /* now we know if it's a file or a dir or maybe something failed */ diff --git a/apps/plugins/rockboy/rockboy.c b/apps/plugins/rockboy/rockboy.c index 46a0aa56c3..93bd98ca78 100644 --- a/apps/plugins/rockboy/rockboy.c +++ b/apps/plugins/rockboy/rockboy.c @@ -71,11 +71,11 @@ void setoptions (void) DIR* dir; char optionsave[sizeof(savedir)+sizeof(optionname)]; - dir=opendir(savedir); + dir=rb->opendir(savedir); if(!dir) - mkdir(savedir); + rb->mkdir(savedir); else - closedir(dir); + rb->closedir(dir); snprintf(optionsave, sizeof(optionsave), "%s/%s", savedir, optionname); diff --git a/apps/plugins/rockboy/rockmacros.h b/apps/plugins/rockboy/rockmacros.h index ecf8a1ef6a..5d60d3f3e3 100644 --- a/apps/plugins/rockboy/rockmacros.h +++ b/apps/plugins/rockboy/rockmacros.h @@ -71,22 +71,13 @@ void dynamic_recompile (struct dynarec_block *newblock); #define isalnum(c) (isdigit(c) || (isalpha(c))) #ifdef SIMULATOR -#undef opendir -#define opendir(a) rb->sim_opendir((a)) -#undef closedir -#define closedir(a) rb->sim_closedir((a)) -#undef mkdir -#define mkdir(a) rb->sim_mkdir((a)) #undef open #define open(a,b) rb->sim_open((a),(b)) -#undef close -#define close(a) rb->close((a)) #undef lseek #define lseek(a,b,c) rb->sim_lseek((a),(b),(c)) +#undef close +#define close(a) rb->close((a)) #else /* !SIMULATOR */ -#define opendir(a) rb->opendir((a)) -#define closedir(a) rb->closedir((a)) -#define mkdir(a) rb->mkdir((a)) #define open(a,b) rb->open((a),(b)) #define lseek(a,b,c) rb->lseek((a),(b),(c)) #define close(a) rb->close((a)) diff --git a/apps/plugins/rockpaint.c b/apps/plugins/rockpaint.c index d478bf9947..a273e4ca37 100644 --- a/apps/plugins/rockpaint.c +++ b/apps/plugins/rockpaint.c @@ -688,7 +688,7 @@ static bool browse( char *dst, int dst_size, const char *start ) while( 1 ) { - d = rb->PREFIX(opendir)( bbuf ); + d = rb->opendir( bbuf ); if( !d ) { /* @@ -702,7 +702,7 @@ static bool browse( char *dst, int dst_size, const char *start ) else if( errno == EACCES || errno == ENOENT ) { bbuf[0] = '/'; bbuf[1] = '\0'; - d = rb->PREFIX(opendir)( "/" ); + d = rb->opendir( "/" ); } else { @@ -714,12 +714,12 @@ static bool browse( char *dst, int dst_size, const char *start ) li = -1; while( i < fvi ) { - rb->PREFIX(readdir)( d ); + rb->readdir( d ); i++; } while( top_inside+(i-fvi)*(fh+LINE_SPACE) < HEIGHT ) { - de = rb->PREFIX(readdir)( d ); + de = rb->readdir( d ); if( !de ) { li = i-1; @@ -737,12 +737,12 @@ static bool browse( char *dst, int dst_size, const char *start ) lvi = i-1; if( li == -1 ) { - if( !rb->PREFIX(readdir)( d ) ) + if( !rb->readdir( d ) ) { li = lvi; } } - rb->PREFIX(closedir)( d ); + rb->closedir( d ); rb->lcd_update(); @@ -863,7 +863,7 @@ static bool browse_fonts( char *dst, int dst_size ) { b_need_redraw = 0; - d = rb->PREFIX(opendir)( FONT_DIR "/" ); + d = rb->opendir( FONT_DIR "/" ); if( !d ) { return false; @@ -873,7 +873,7 @@ static bool browse_fonts( char *dst, int dst_size ) li = -1; while( i < fvi ) { - rb->PREFIX(readdir)( d ); + rb->readdir( d ); i++; } cp = top_inside+LINE_SPACE; @@ -883,7 +883,7 @@ static bool browse_fonts( char *dst, int dst_size ) while( cp < top+HEIGHT ) { - de = rb->PREFIX(readdir)( d ); + de = rb->readdir( d ); if( !de ) { li = i-1; @@ -920,7 +920,7 @@ static bool browse_fonts( char *dst, int dst_size ) lvi = i-1; if( li == -1 ) { - if( !(de = rb->PREFIX(readdir)( d ) ) ) + if( !(de = rb->readdir( d ) ) ) { li = lvi; } @@ -936,7 +936,7 @@ static bool browse_fonts( char *dst, int dst_size ) } } rb->font_load( old_font ); - rb->PREFIX(closedir)( d ); + rb->closedir( d ); } rb->lcd_set_drawmode(DRMODE_COMPLEMENT); -- cgit v1.2.3