[Kde-bindings] Smoke::numMethodNames of by one
Richard Dale
rdale at foton.es
Tue Dec 9 15:42:25 UTC 2008
On Tuesday 09 December 2008 13:41:06 Tobias Rautenkranz wrote:
> You are right -- idMethodName requires numMethodNames to be the index of
> the last element (not the number of elements).
>
> It is however inconsistent with numClasses, which is the number of
> elements.
>
> But this works out since there is an:
> > if (icur > 0 && icur < numClasses)
I think this is unnecessary and I removed it after changing numClasses to be
the number of classes.
> in idClass().
>
> Thanks for the reply and please disregard my patch.
I think that numMethodNames is correct and the other ones are wrong, so I've
now changed them all so that 'numThings' at the end of smokedata.cpp is always
the same as the number of things in the table.
-- Richard
More information about the Kde-bindings
mailing list