Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: uchida: r27333 - trunk/apps

Re: uchida: r27333 - trunk/apps

From: Frank Gevaerts <frank_at_gevaerts.be>
Date: Wed, 7 Jul 2010 13:43:30 +0200

Any reason why the new functions weren't added at the end of the struct,
so PLUGIN_MIN_API_VERSION wouldn't have had to increase?

Frank

On Wed, Jul 07, 2010 at 01:35:30PM +0200, mailer_at_svn.rockbox.org wrote:
> Date: 2010-07-07 13:35:30 +0200 (Wed, 07 Jul 2010)
> New Revision: 27333
>
> Log Message:
> plugin api: add the following functions.
> - get the viewport of the current skin.
> - update the current skin statusbar.
>
> Modified:
> trunk/apps/plugin.c
> trunk/apps/plugin.h
>
> Modified: trunk/apps/plugin.c
> ===================================================================
> --- trunk/apps/plugin.c 2010-07-07 09:33:47 UTC (rev 27332)
> +++ trunk/apps/plugin.c 2010-07-07 11:35:30 UTC (rev 27333)
> _at__at_ -565,6 +565,11 _at__at_
> /* statusbars */
> &statusbars,
> gui_syncstatusbar_draw,
> +#ifdef HAVE_LCD_BITMAP
> + sb_skin_get_info_vp,
> + sb_skin_update,
> +#endif
> +
> /* options */
> get_settings_list,
> find_setting,
>
> Modified: trunk/apps/plugin.h
> ===================================================================
> --- trunk/apps/plugin.h 2010-07-07 09:33:47 UTC (rev 27332)
> +++ trunk/apps/plugin.h 2010-07-07 11:35:30 UTC (rev 27333)
> _at__at_ -84,6 +84,7 _at__at_
> #include "scrollbar.h"
> #include "jpeg_load.h"
> #include "../recorder/bmp.h"
> +#include "statusbar-skinned.h"
> #endif
> #include "statusbar.h"
> #include "menu.h"
> _at__at_ -144,12 +145,12 _at__at_
> #define PLUGIN_MAGIC 0x526F634B /* RocK */
>
> /* increase this every time the api struct changes */
> -#define PLUGIN_API_VERSION 188
> +#define PLUGIN_API_VERSION 189
>
> /* 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 188
> +#define PLUGIN_MIN_API_VERSION 189
>
> /* plugin return codes */
> enum plugin_status {
> _at__at_ -694,6 +695,10 _at__at_
> /* scroll bar */
> struct gui_syncstatusbar *statusbars;
> void (*gui_syncstatusbar_draw)(struct gui_syncstatusbar * bars, bool force_redraw);
> +#ifdef HAVE_LCD_BITMAP
> + struct viewport *(*sb_skin_get_info_vp)(enum screen_type screen);
> + void (*sb_skin_update)(enum screen_type screen, bool force);
> +#endif
>
> /* options */
> const struct settings_list* (*get_settings_list)(int*count);
>
> _______________________________________________
> rockbox-cvs mailing list
> rockbox-cvs_at_cool.haxx.se
> http://cool.haxx.se/cgi-bin/mailman/listinfo/rockbox-cvs

-- 
"Debugging is twice as hard as writing the code in the first place.
Therefore, if you write the code as cleverly as possible, you are,
by definition, not smart enough to debug it." - Brian W. Kernighan
Received on 2010-07-07

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy