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