[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