[Kde-bindings] Adding an mf_attribute flag to MethodFlags in smoke.h

[Arno Rehn]

On Saturday 24 October 2009 18:42:27 Arno Rehn wrote:
> On Wednesday 21 October 2009 02:59:07 Arno Rehn wrote:
> > On Tuesday 13 October 2009 13:55:51 Arno Rehn wrote:
> > > On Tuesday 13 October 2009 12:38:33 Richard Dale wrote:
> > > > Ah, after I posted the mail I realized the flags are in an unsigned
> > > > char, and we've run out of space. So it would need to be changed to
> > > > an unsigned short instead. Maybe wait until we have several BIC
> > > > changes to smoke.h and do them all at once? Now we have several
> > > > projects using the Smoke libs we have to be careful to all keep using
> > > > the same versions.
> > >
> > > If we're at it we could also add sizeof() information for classes (for
> > >  dynamic list marshallers) and make init_foo_Smoke() and foo_Smoke
> > > extern "C" (as suggested by the lisp bindings developer).
> > >
> > > > Also I think a 'mf_overloaded' flag would be useful as if a method
> > > > isn't overloaded, there is no need to keep looking it up, and the
> > > > result of the first lookup can be cached.
> > >
> > > Yes, that sounds good.
> >
> > As the soft feature freeze is getting closer (only 2 weeks left), I'd
> > like to break BC before that or not until trunk is open for KDE 4.5, so
> > people developing with Smoke (and that number seems to be steadily
> > increasing :)) and packagers have time to adjust to the new version.
> > Personally I'd opt for breaking it now, so to not hinder development of
> > new stuff. If everyone is fine with that, we could collect a list of
> > things that should go in. Until now I see the following:
> >
> > * The above discussed extension to method flags.
> > * Mark init_foo_Smoke and foo_Smoke extern "C".
> > * Add sizeof() information to the classes table.
> >
> > Would this be ok? Is there anything you would add? Or not include?
> 
> I'd also add a delete_foo_Smoke with an extern "C" binding, so we can
>  easily delete the smoke lib again (for languages that don't support this
>  natively, like C#).
As noone seems to have objections, I'll commit this tonight. I'll split up the 
commit, so if anything isn't ok, we can easily revert.

-- 
Arno Rehn
arno at arnorehn.de