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: othelo (*sigh*)
From: BlueChip (cs_bluechip_at_webtribe.net)
Date: 2003-08-09


At 12:05 09/08/03 +0200, you wrote:

>"BlueChip" <cs_bluechip_at_webtribe.net> wrote in message
>news:5.2.0.9.0.20030809012911.00a47550_at_pop.webtribe.net...
> > At 15:25 08/08/03 -0700, you wrote:
> > >i mentioned this once before, but no one seemed to notice it. othelo
> > >doesn't work on my FMR. at
> > >all. i just get an 'incompatable model' error. what happened? *glares at
> > >bluechip* ;-)
> > >
> > >-scott
> >
> > follow
> >
> >
> > apps/plugins/othelo.c:1352
> > TEST_PLUGIN_API(api);
> >
> >
> > apps/plugin.h:75
> > #define TEST_PLUGIN_API(_api_) \
> > do { \
> > int _rc_ = _api_->plugin_test(PLUGIN_API_VERSION, MODEL, MEM); \
> > if (_rc_<0) \
> > return _rc_; \
> > } while(0)
> >
> >
> > apps/plugin.c:279
> > static int plugin_test(int api_version, int model, int memsize)
> > {
> > if (api_version < PLUGIN_MIN_API_VERSION ||
> > api_version > PLUGIN_API_VERSION)
> > return PLUGIN_WRONG_API_VERSION;
> > if (model != MODEL)
> > return PLUGIN_WRONG_MODEL;
> > if (memsize != MEM)
> > return PLUGIN_WRONG_MODEL;
> > return PLUGIN_OK;
> > }
> >
> >
> > ...QED, the bug is in the plugin code, or the way it is called.
> >
> > The error is generated by rockbox before othelo has even had a chance to
> > execute it's very first line of code!
> >
> > Maybe you are running a version of othelo that was compiled for the
>recorder?
> >
> >
>
>No your wrong, the code you are quoting refuses to load your plugin because
>it has an incompatible version
>of the plugin interface. The othelo code produces a othelo.rock with file
>size zero and is therfore not uploded in the daily rocks.
>So the latest good othelo compile is incompatible with the current rockbox
>plugin interface.
>
>If you move the inlude of plugin.h to befor the ifdef HAVE_LCD_BITMAP in
>YOUR code it will
>work again.
>
> #include "plugin.h"
>#ifdef HAVE_LCD_BITMAP
>
>
>
>.../Henrikl

That bug was fixed by "nix" on 1st August, I did not realise that it was
never included into the daily builds.



Page was last modified "Jan 10 2012" The Rockbox Crew
aaa