[pLog-svn] r4171 - plugins/branches/lifetype-1.1/gallery

[Oscar Renalias]

I think the PluginBase class should only include the classes that are
strictly necessary and the Menu class wasn't one of them. I don't
think it's too much work right now to check which plugins need it and
add an extra include.

On 10/24/06, Reto Hugi <plog at hugi.to> wrote:
> On 24.10.2006 16:45, Oscar Renalias wrote:
> > It probably is needed by most of the other plugins but you don't
> > notice because as soon as one of them loads the class, it becomes
> > automatically available to all other plugins.
> >
> > But if you test plugins one by one (install, test, remove), then
> > you're probably going to find a few more cases like this. It probably
> > is just a matter of going through all of them, check if they require
> > the Menu class and if they do, add the include at the top.
> >
>
> right, of course! isn't this something we should include in the
> pluginbase? this could save some overhead checking includes...
>
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.lifetype.net
> http://devel.lifetype.net/mailman/listinfo/plog-svn
>