4 files changed, 97 insertions, 125 deletions

diff --git a/apps/plugin.c b/apps/plugin.c
index 413caf87a1..e5a496c567 100644
--- a/apps/plugin.c
+++ b/apps/plugin.c
@@ -88,19 +88,25 @@ static const struct plugin_api rockbox_api = {
     lcd_puts,
     lcd_puts_scroll,
     lcd_stop_scroll,
+    lcd_set_contrast,
 #ifdef HAVE_LCD_CHARCELLS
     lcd_define_pattern,
     lcd_get_locked_pattern,
     lcd_unlock_pattern,
     lcd_putc,
+    lcd_put_cursor,
+    lcd_remove_cursor,
+    lcd_icon,
 #else
     lcd_putsxy,
+    lcd_puts_style,
     lcd_bitmap,
     lcd_drawline,
     lcd_clearline,
     lcd_drawpixel,
     lcd_clearpixel,
     lcd_setfont,
+    font_get,
     lcd_clearrect,
     lcd_fillrect,
     lcd_drawrect,
@@ -109,14 +115,21 @@ static const struct plugin_api rockbox_api = {
     lcd_update,
     lcd_update_rect,
     scrollbar,
+    checkbox,
+    &lcd_framebuffer[0][0],
+    lcd_blit,
 #ifndef SIMULATOR
     lcd_roll,
 #endif
 #endif
+    backlight_on,
+    backlight_off,
 
     /* button */
     button_get,
     button_get_w_tmo,
+    button_status,
+    button_clear_queue,
 
     /* file */
     (open_func)PREFIX(open),
@@ -131,6 +144,7 @@ static const struct plugin_api rockbox_api = {
     PREFIX(filesize),
     fprintf,
     read_line,
+    settings_parseline,
 
     /* dir */
     PREFIX(opendir),
@@ -139,14 +153,21 @@ static const struct plugin_api rockbox_api = {
 
     /* kernel */
     PREFIX(sleep),
+    yield,
     usb_screen,
     &current_tick,
+    default_event_handler,
+    create_thread,
+    remove_thread,
 
     /* strings and memory */
     snprintf,
     strcpy,
+    strncpy,
     strlen,
     strrchr,
+    strcmp,
+    strcasecmp,
     memset,
     memcpy,
 #ifndef SIMULATOR
@@ -154,12 +175,31 @@ static const struct plugin_api rockbox_api = {
 #endif
 
     /* sound */
+    mpeg_sound_set,
 #ifndef SIMULATOR
+    mp3_play_data,
+    mp3_play_pause,
+    mp3_play_stop,
+    mp3_is_playing,
+    bitswap,
 #ifdef HAVE_MAS3587F
     mas_codec_readreg,
 #endif
 #endif
     
+    /* playback control */
+    mpeg_play,
+    mpeg_stop,
+    mpeg_pause,
+    mpeg_resume,
+    mpeg_next,
+    mpeg_prev,
+    mpeg_ff_rewind,
+    mpeg_next_track,
+    playlist_amount,
+    mpeg_status,
+    mpeg_has_changed_track,
+
     /* misc */
     srand,
     rand,
@@ -170,54 +210,21 @@ static const struct plugin_api rockbox_api = {
     atoi,
     get_time,
     plugin_get_buffer,
+    plugin_get_mp3_buffer,
+#ifndef SIMULATOR
+    plugin_register_timer,
+    plugin_unregister_timer,
+#endif
+    plugin_tsr,
 
     /* new stuff at the end, sort into place next time the API gets incompatible */
 
-#ifndef HAVE_LCD_CHARCELLS
-    &lcd_framebuffer[0][0],
-    lcd_blit,
-#endif
-    yield,
 
-    plugin_get_mp3_buffer,
-    mpeg_sound_set,
-#ifndef SIMULATOR
-    mp3_play_data,
-    mp3_play_pause,
-    mp3_play_stop,
-    mp3_is_playing,
-    bitswap,
-#endif
     &global_settings,
     backlight_set_timeout,
 #ifndef SIMULATOR
     ata_sleep,
 #endif
-#ifdef HAVE_LCD_BITMAP
-    checkbox,
-#endif
-#ifndef SIMULATOR
-    plugin_register_timer,
-    plugin_unregister_timer,
-#endif
-    plugin_tsr,
-    create_thread,
-    remove_thread,
-    lcd_set_contrast,
-    mpeg_play,
-    mpeg_stop,
-    mpeg_pause,
-    mpeg_resume,
-    mpeg_next,
-    mpeg_prev,
-    mpeg_ff_rewind,
-    mpeg_next_track,
-    playlist_amount,
-    mpeg_status,
-    mpeg_has_changed_track,
-#ifdef HAVE_LCD_BITMAP
-    font_get,
-#endif
 #if defined(DEBUG) || defined(SIMULATOR)
     debugf,
 #endif
@@ -238,26 +245,6 @@ static const struct plugin_api rockbox_api = {
     set_time,
     reset_poweroff_timer,
 
-    backlight_on,
-    backlight_off,
-
-#ifdef HAVE_LCD_CHARCELLS
-    lcd_icon,
-#endif
-#ifdef HAVE_LCD_BITMAP
-    lcd_puts_style,
-#endif
-#ifdef HAVE_LCD_CHARCELLS
-    lcd_put_cursor,
-    lcd_remove_cursor,
-#endif
-    settings_parseline,
-    strcmp,
-    button_status,
-    button_clear_queue,
-    strncpy,
-    strcasecmp,
-    default_event_handler
 };
 
 int plugin_load(char* plugin, void* parameter)
diff --git a/apps/plugin.h b/apps/plugin.h
index 74700879b4..51607a53e6 100644
--- a/apps/plugin.h
+++ b/apps/plugin.h
@@ -60,12 +60,12 @@
 #endif
 
 /* increase this every time the api struct changes */
-#define PLUGIN_API_VERSION 24
+#define PLUGIN_API_VERSION 25
 
 /* update this to latest version if a change to the api struct breaks
    backwards compatibility (and please take the opportunity to sort in any 
    new function which are "waiting" at the end of the function table) */
-#define PLUGIN_MIN_API_VERSION 19
+#define PLUGIN_MIN_API_VERSION 25
 
 /* plugin return codes */
 enum plugin_status {
@@ -114,13 +114,18 @@ struct plugin_api {
     void (*lcd_puts)(int x, int y, unsigned char *string);
     void (*lcd_puts_scroll)(int x, int y, unsigned char* string);
     void (*lcd_stop_scroll)(void);
+    void (*lcd_set_contrast)(int x);
 #ifdef HAVE_LCD_CHARCELLS
     void (*lcd_define_pattern)(int which,char *pattern);
     unsigned char (*lcd_get_locked_pattern)(void);
     void (*lcd_unlock_pattern)(unsigned char pat);
     void (*lcd_putc)(int x, int y, unsigned short ch);
+    void (*lcd_put_cursor)(int x, int y, char cursor_char);
+    void (*lcd_remove_cursor)(void);
+    void (*lcd_icon)(int icon, bool enable);
 #else
     void (*lcd_putsxy)(int x, int y, const unsigned char *string);
+    void (*lcd_puts_style)(int x, int y, unsigned char *str, int style);
     void (*lcd_bitmap)(const unsigned char *src, int x, int y,
                        int nx, int ny, bool clear);
     void (*lcd_drawline)(int x1, int y1, int x2, int y2);
@@ -128,6 +133,7 @@ struct plugin_api {
     void (*lcd_drawpixel)(int x, int y);
     void (*lcd_clearpixel)(int x, int y);
     void (*lcd_setfont)(int font);
+    struct font* (*font_get)(int font);
     void (*lcd_clearrect)(int x, int y, int nx, int ny);
     void (*lcd_fillrect)(int x, int y, int nx, int ny);
     void (*lcd_drawrect)(int x, int y, int nx, int ny);
@@ -137,14 +143,21 @@ struct plugin_api {
     void (*lcd_update_rect)(int x, int y, int width, int height);
     void (*scrollbar)(int x, int y, int width, int height, int items,
                       int min_shown, int max_shown, int orientation);
+    void (*checkbox)(int x, int y, int width, int height, bool checked);
+    unsigned char* lcd_framebuffer;
+    void (*lcd_blit) (unsigned char* p_data, int x, int y, int width, int height, int stride);
 #ifndef SIMULATOR
     void (*lcd_roll)(int pixels);
 #endif
 #endif
+    void (*backlight_on)(void);
+    void (*backlight_off)(void);
 
     /* button */
     int (*button_get)(bool block);
     int (*button_get_w_tmo)(int ticks);
+    int (*button_status)(void);
+    void (*button_clear_queue)(void);
 
     /* file */
     int (*open)(const char* pathname, int flags);
@@ -159,6 +172,7 @@ struct plugin_api {
     int (*filesize)(int fd);
     int (*fprintf)(int fd, const char *fmt, ...);
     int (*read_line)(int fd, char* buffer, int buffer_size);
+    bool (*settings_parseline)(char* line, char** name, char** value);
     
     /* dir */
     DIR* (*opendir)(const char* name);
@@ -167,14 +181,21 @@ struct plugin_api {
 
     /* kernel */
     void (*sleep)(int ticks);
+    void (*yield)(void);
     void (*usb_screen)(void);
     long* current_tick;
+    int (*default_event_handler)(int event);
+    int (*create_thread)(void* function, void* stack, int stack_size, char *name);
+    void (*remove_thread)(int threadnum);
 
     /* strings and memory */
     int    (*snprintf)(char *buf, size_t size, const char *fmt, ...);
     char*  (*strcpy)(char *dst, const char *src);
+    char *(*strncpy)(char *dst, const char *src, size_t length);
     size_t (*strlen)(const char *str);
     char * (*strrchr)(const char *s, int c);
+    int (*strcmp)(const char *, const char *);
+    int (*strcasecmp)(const char *, const char *);
     void*  (*memset)(void *dst, int c, size_t length);
     void*  (*memcpy)(void *out, const void *in, size_t n);
 #ifndef SIMULATOR
@@ -182,12 +203,31 @@ struct plugin_api {
 #endif
 
     /* sound */
+    void (*mpeg_sound_set)(int setting, int value);
 #ifndef SIMULATOR
+    void (*mp3_play_data)(unsigned char* start, int size, void (*get_more)(unsigned char** start, int* size));
+    void (*mp3_play_pause)(bool play);
+    void (*mp3_play_stop)(void);
+    bool (*mp3_is_playing)(void);
+    void (*bitswap)(unsigned char *data, int length);
 #ifdef HAVE_MAS3587F
     int (*mas_codec_readreg)(int reg);
 #endif
 #endif
 
+    /* playback control */
+    void (*mpeg_play)(int offset);
+    void (*mpeg_stop)(void);
+    void (*mpeg_pause)(void);
+    void (*mpeg_resume)(void);
+    void (*mpeg_next)(void);
+    void (*mpeg_prev)(void);
+    void (*mpeg_ff_rewind)(int newtime);
+    struct mp3entry* (*mpeg_next_track)(void);
+    int (*playlist_amount)(void);
+    int (*mpeg_status)(void);
+    bool (*mpeg_has_changed_track)(void);
+    
     /* misc */
     void (*srand)(unsigned int seed);
     int  (*rand)(void);
@@ -199,61 +239,26 @@ struct plugin_api {
     int (*atoi)(const char *str);
     struct tm* (*get_time)(void);
     void* (*plugin_get_buffer)(int* buffer_size);
-
-    /* new stuff, sort in next time the API gets broken! */
-#ifndef HAVE_LCD_CHARCELLS
-    unsigned char* lcd_framebuffer;
-    void (*lcd_blit) (unsigned char* p_data, int x, int y, int width, int height, int stride);
-#endif
-    void (*yield)(void);
-
     void* (*plugin_get_mp3_buffer)(int* buffer_size);
-    void (*mpeg_sound_set)(int setting, int value);
 #ifndef SIMULATOR
-    void (*mp3_play_data)(unsigned char* start, int size, void (*get_more)(unsigned char** start, int* size));
-    void (*mp3_play_pause)(bool play);
-    void (*mp3_play_stop)(void);
-    bool (*mp3_is_playing)(void);
-    void (*bitswap)(unsigned char *data, int length);
+    int (*plugin_register_timer)(int cycles, int prio, void (*timer_callback)(void));
+    void (*plugin_unregister_timer)(void);
 #endif
+    void (*plugin_tsr)(void (*exit_callback)(void));
+
+    /* new stuff, sort in next time the API gets broken! */
+
     struct user_settings* global_settings;
     void (*backlight_set_timeout)(int index);
 #ifndef SIMULATOR
     int (*ata_sleep)(void);
 #endif
-#ifdef HAVE_LCD_BITMAP
-    void (*checkbox)(int x, int y, int width, int height, bool checked);
-#endif
-#ifndef SIMULATOR
-    int (*plugin_register_timer)(int cycles, int prio, void (*timer_callback)(void));
-    void (*plugin_unregister_timer)(void);
-#endif
-    void (*plugin_tsr)(void (*exit_callback)(void));
-    int (*create_thread)(void* function, void* stack, int stack_size, char *name);
-    void (*remove_thread)(int threadnum);
-    void (*lcd_set_contrast)(int x);
 
-    /* playback control */
-    void (*mpeg_play)(int offset);
-    void (*mpeg_stop)(void);
-    void (*mpeg_pause)(void);
-    void (*mpeg_resume)(void);
-    void (*mpeg_next)(void);
-    void (*mpeg_prev)(void);
-    void (*mpeg_ff_rewind)(int newtime);
-    struct mp3entry* (*mpeg_next_track)(void);
-    int (*playlist_amount)(void);
-    int (*mpeg_status)(void);
-    bool (*mpeg_has_changed_track)(void);
-    
-#ifdef HAVE_LCD_BITMAP
-    struct font* (*font_get)(int font);
-#endif
 
 #if defined(DEBUG) || defined(SIMULATOR)
    void (*debugf)(char *fmt, ...);
 #endif
-   bool (*mp3info)(struct mp3entry *entry, char *filename) ;
+   bool (*mp3info)(struct mp3entry *entry, char *filename, bool v1first);
    int (*count_mp3_frames)(int fd, int startpos, int filesize,
                            void (*progressfunc)(int));
    int (*create_xing_header)(int fd, int startpos, int filesize,
@@ -274,26 +279,6 @@ struct plugin_api {
     int  (*set_time)(struct tm *tm);
     void (*reset_poweroff_timer)(void);
 
-    void (*backlight_on)(void);
-    void (*backlight_off)(void);
-
-#ifdef HAVE_LCD_CHARCELLS
-    void (*lcd_icon)(int icon, bool enable);
-#endif
-#ifdef HAVE_LCD_BITMAP
-    void (*lcd_puts_style)(int x, int y, unsigned char *str, int style);
-#endif
-#ifdef HAVE_LCD_CHARCELLS
-    void (*lcd_put_cursor)(int x, int y, char cursor_char);
-    void (*lcd_remove_cursor)(void);
-#endif
-    bool (*settings_parseline)(char* line, char** name, char** value);
-    int (*strcmp)(const char *, const char *);
-    int (*button_status)(void);
-    void (*button_clear_queue)(void);
-    char *(*strncpy)(char *dst, const char *src, size_t length);
-    int (*strcasecmp)(const char *, const char *);
-    int (*default_event_handler)(int event);
 };
 
 /* defined by the plugin loader (plugin.c) */
diff --git a/apps/plugins/vbrfix.c b/apps/plugins/vbrfix.c
index 1beca2669d..0077340ec9 100644
--- a/apps/plugins/vbrfix.c
+++ b/apps/plugins/vbrfix.c
@@ -145,7 +145,7 @@ static bool vbr_fix(char *selected_file)
 
     xingupdate(0);
 
-    rc = rb->mp3info(&entry, selected_file);
+    rc = rb->mp3info(&entry, selected_file, false);
     if(rc < 0) {
         fileerror(rc);
         return true;
diff --git a/apps/talk.c b/apps/talk.c
index 462ab85dd1..51e8ae7f01 100644
--- a/apps/talk.c
+++ b/apps/talk.c
@@ -390,7 +390,7 @@ int talk_file(char* filename, bool enqueue)
     if (p_thumbnail == NULL || size_for_thumbnail <= 0)
         return -1;
 
-    if(mp3info(&info, filename)) /* use this to find real start */
+    if(mp3info(&info, filename, false)) /* use this to find real start */
     {   
         return 0; /* failed to open, or invalid */
     }