Cleanup of KDevPlugin and other interfaces

Matt Rogers mattr at kde.org
Mon Nov 1 23:32:02 UTC 2004


On Mon, Nov 01, 2004 at 10:22:18PM +0000, Richard Dale wrote:
> On Monday 01 November 2004 21:45, Alexander Dymo wrote:
> > I have one proposal on how to cleanup our interfaces. We can use
> > KDevPlugin::extension() methods to reduce the number of methods
> > and interfaces we have (look at the patch). The idea is to provide a way
> > to query for any extension which might be available.
> >
> > My patch removes all stuff about source formatter from interfaces.
> > Patch for cpp support shows how extensions can be used.
> > extensions.tar.bz2 should be extracted into lib/interfaces dir.
> > Other suggestions are in attachment.
> Can I add this suggestion - a Package access type for java, and using enums 
> for access in the interface:
> 
> class CodeModelItem: public KShared
> {
> public:
> ...
>     enum
>     {
>  Public,
>  Protected,
>  Private,
>         Package  // Needed for java
>     };
> 
> class FunctionModel: public CodeModelItem
> {
> ...
>     int access() const;
>     void setAccess( int access );
> 
> Shouldn't these be an enum rather than ints?
> 

If you give the enums a typename, i think they can be made enums,
otherwise (and AFAIK) they have to be treated as ints.

Matt




More information about the KDevelop-devel mailing list