Revert my filetypes commits from today. the nvram buffer is 44bytes of which 43 are currently being used. We need to figure out what to do about this.

So in the mean time, set the MAX_FILETYPES to a huge value so we don't have to worry about this for a long while.

This build shold be safe to use again. If you downloaded a build after r14286 make sure you grab a new one


git-svn-id: svn://svn.rockbox.org/rockbox/trunk@14291 a1c6a512-1295-4272-9138-f99709370657

1 files changed, 25 insertions, 37 deletions

diff --git a/apps/filetypes.c b/apps/filetypes.c
index 090898a3af..21ce17ce40 100644
--- a/apps/filetypes.c
+++ b/apps/filetypes.c
@@ -41,12 +41,20 @@
 #include "icons.h"
 #include "logf.h"
 
+/* max filetypes (plugins & icons stored here) */
+#if CONFIG_CODEC == SWCODEC
+#define MAX_FILETYPES 128
+#else
+#define MAX_FILETYPES 48
+#endif
+
 /* a table for the know file types */
 const struct filetype inbuilt_filetypes[] = {
     { "mp3", FILE_ATTR_AUDIO, Icon_Audio, VOICE_EXT_MPA },
     { "mp2", FILE_ATTR_AUDIO, Icon_Audio, VOICE_EXT_MPA },
     { "mpa", FILE_ATTR_AUDIO, Icon_Audio, VOICE_EXT_MPA },
 #if CONFIG_CODEC == SWCODEC
+    /* Temporary hack to allow playlist creation */
     { "mp1", FILE_ATTR_AUDIO, Icon_Audio, VOICE_EXT_MPA },
     { "ogg", FILE_ATTR_AUDIO, Icon_Audio, VOICE_EXT_MPA },
     { "wma", FILE_ATTR_AUDIO, Icon_Audio, VOICE_EXT_MPA },
@@ -96,7 +104,7 @@ const struct filetype inbuilt_filetypes[] = {
     { "cue",  FILE_ATTR_CUE,   Icon_Bookmark,  VOICE_EXT_CUESHEET },
 #ifdef BOOTFILE_EXT
     { BOOTFILE_EXT, FILE_ATTR_MOD, Icon_Firmware, VOICE_EXT_AJZ },
-#endif /* #ifndef BOOTFILE_EXT */
+#endif /* #ifndef SIMULATOR */
 };
 
 void tree_get_filetypes(const struct filetype** types, int* count)
@@ -115,14 +123,14 @@ struct file_type {
     char* plugin; /* Which plugin to use, NULL if unknown, or builtin */
     char* extension; /* NULL for none */
 };
-static struct file_type *filetypes;
-static int *custom_filetype_icons;
+static struct file_type filetypes[MAX_FILETYPES];
+static int custom_filetype_icons[MAX_FILETYPES];
 static bool custom_icons_loaded = false;
 #ifdef HAVE_LCD_COLOR
-static int *custom_colors;
+static int custom_colors[MAX_FILETYPES+1];
 #endif
+static int filetype_count = 0;
 static unsigned char heighest_attr = 0;
-static int max_types = 0, filetype_count;
 
 static char *filetypes_strdup(char* string)
 {
@@ -142,10 +150,7 @@ void read_color_theme_file(void) {
     int fd;
     char *ext, *color;
     int i;
-
-    if (max_types == 0)
-        return;
-    for (i = 0; i < filetype_count+1; i++) {
+    for (i = 0; i < MAX_FILETYPES+1; i++) {
         custom_colors[i] = -1;
     }
     snprintf(buffer, MAX_PATH, "%s/%s.colours", THEME_DIR, 
@@ -164,7 +169,7 @@ void read_color_theme_file(void) {
         }
         if (!strcasecmp(ext, "???"))
         {
-            custom_colors[filetype_count] = hex_to_rgb(color);
+            custom_colors[MAX_FILETYPES] = hex_to_rgb(color);
             continue;
         }
         for (i=1; i<filetype_count; i++)
@@ -190,8 +195,6 @@ void read_viewer_theme_file(void)
     global_status.viewer_icon_count = 0;
     custom_icons_loaded = false;
     custom_filetype_icons[0] = Icon_Folder;
-    if (max_types == 0)
-        return;
     for (i=1; i<filetype_count; i++)
     {
         custom_filetype_icons[i] = filetypes[i].icon;
@@ -233,16 +236,6 @@ void read_viewer_theme_file(void)
 
 void  filetype_init(void)
 {
-    max_types = global_status.filetype_count + 8; /* always make a bit more room
-                                                     for more types */
-    filetypes = (struct file_type *)buffer_alloc(sizeof(struct file_type)*
-                                                 max_types);
-    custom_filetype_icons = (int*)buffer_alloc(sizeof(int)*max_types);
-#ifdef HAVE_LCD_COLOR
-    /* the extra item here is for the unknown types
-       which use the last array element */
-    custom_colors = (int*)buffer_alloc(sizeof(int)*(max_types+1));
-#endif
     /* set the directory item first */
     filetypes[0].extension = NULL;
     filetypes[0].plugin = NULL;
@@ -258,10 +251,6 @@ void  filetype_init(void)
 #ifdef HAVE_LCD_COLOR
     read_color_theme_file();
 #endif
-    if (global_status.filetype_count == 0)
-        global_status.filetype_count = MAX_FILETYPES;
-    else global_status.filetype_count = filetype_count;
-    status_save();
 }
 
 /* remove all white spaces from string */
@@ -283,14 +272,14 @@ static void rm_whitespaces(char* str)
 static void read_builtin_types(void)
 {
     int count = sizeof(inbuilt_filetypes)/sizeof(*inbuilt_filetypes), i;
-    for(i=0; i<count && (filetype_count<max_types); i++)
+    for(i=0; i<count && (filetype_count < MAX_FILETYPES); i++)
     {
         filetypes[filetype_count].extension = inbuilt_filetypes[i].extension;
         filetypes[filetype_count].plugin = NULL;
-        filetypes[filetype_count].attr = inbuilt_filetypes[i].tree_attr>>8;
+        filetypes[filetype_count].attr   = inbuilt_filetypes[i].tree_attr>>8;
         if (filetypes[filetype_count].attr > heighest_attr)
             heighest_attr = filetypes[filetype_count].attr;
-        filetypes[filetype_count].icon = inbuilt_filetypes[i].icon;
+        filetypes[filetype_count].icon   = inbuilt_filetypes[i].icon;
         filetype_count++;
     }
 }
@@ -302,15 +291,14 @@ static void read_config(char* config_file)
     int fd = open(config_file, O_RDONLY);
     if (fd < 0)
         return;
-    /* config file is in the form 
+    /* config file is in the for 
        <extension>,<plugin>,<icon code>
        ignore line if either of the first two are missing */
     while (read_line(fd, line, 64) > 0)
     {
-        if (filetype_count >= max_types)
+        if (filetype_count >= MAX_FILETYPES)
         {
             gui_syncsplash(HZ, ID2P(LANG_FILETYPES_FULL));
-            global_status.filetype_count = 0; /* make plenty of room for next reboot */
             break;
         }
         rm_whitespaces(line);
@@ -359,7 +347,7 @@ int filetype_get_attr(const char* file)
     if (!extension)
         return 0;
     extension++;
-    for (i=0; i<max_types; i++)
+    for (i=0; i<filetype_count; i++)
     {
         if (filetypes[i].extension && 
             !strcasecmp(extension, filetypes[i].extension))
@@ -391,7 +379,7 @@ int filetype_get_color(const char * name, int attr)
         return custom_colors[0];
     extension = strrchr(name, '.');
     if (!extension)
-        return custom_colors[filetype_count];
+        return custom_colors[MAX_FILETYPES];
     extension++;
     
     for (i=1; i<filetype_count; i++)
@@ -400,7 +388,7 @@ int filetype_get_color(const char * name, int attr)
             !strcasecmp(extension, filetypes[i].extension))
             return custom_colors[i];
     }
-    return custom_colors[filetype_count];
+    return custom_colors[MAX_FILETYPES];
 }
 #endif
 
@@ -451,9 +439,9 @@ char * openwith_get_name(int selected_item, void * data, char * buffer)
 int filetype_list_viewers(const char* current_file)
 {
     int i, count = 0, action;
-    int items[64];
+    int items[MAX_FILETYPES];
     struct gui_synclist lists;
-    for (i=0; i<filetype_count && count < 64; i++)
+    for (i=0; i<filetype_count && count < MAX_FILETYPES; i++)
     {
         if (filetypes[i].plugin)
         {