Mailing List archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[vdr] Re: VDR Plugin System - First Step



Stefan Huelswitt wrote:
> 
> On 09 May 2002 Klaus Schmidinger <Klaus.Schmidinger@cadsoft.de> wrote:
> 
> > I suggest that the authors of VDR patches try to implement the "command line",
> > "menu", "setup" and "i18n" parts of their patches, using the new plugin features, and
> 
> And one more: in tools.h:
> 
> @@ -147,8 +149,8 @@
>    T *Get(int Index) const { return (T *)cListBase::Get(Index); }
>    T *First(void) const { return (T *)objects; }
>    T *Last(void) const { return (T *)lastObject; }
> -  T *Prev(const T *object) const { return (T *)object->Prev(); }
> -  T *Next(const T *object) const { return (T *)object->Next(); }
> +  T *Prev(const T *object) const { return (T *)object->cListObject::Prev(); }
> +  T *Next(const T *object) const { return (T *)object->cListObject::Next(); }
>    };
> 
> else compiling fails for classes which are ListObject and
> ListBase at one. e.g.:
> 
> class cPlayList : public cConfig<cMP3>, public cListObject {
>   };

Is that really a good idea to derive cPlayList from cConfig *and* cListObject?
Shouldn't cPlayList rather have a *member* of type cConfig?

Klaus
-- 
_______________________________________________________________

Klaus Schmidinger                       Phone: +49-8635-6989-10
CadSoft Computer GmbH                   Fax:   +49-8635-6989-40
Hofmark 2                               Email:   kls@cadsoft.de
D-84568 Pleiskirchen, Germany           URL:     www.cadsoft.de
_______________________________________________________________



Home | Main Index | Thread Index