[PATCH] ***FINAL*** API review of KCalendarSystem* and KDate* widgets
David Jarvie
lists at astrojar.org.uk
Mon Jul 23 11:34:04 BST 2007
On Monday 23 July 2007 11:26, Alex Merry wrote:
> On Monday 23 Jul 2007, John Layt wrote:
>> P.S. Quick question, pardon my ignorance on C++ & BIC, but does
>> re-arranging the order of methods in the header change the virtual
>> table? I want a more logical grouping of methods for the apidox, but
>> doing so now makes the patches very hard to review.
>
> Only if you're changing the order of virtual methods.
You could always apply a separate patch to alter the method order, after
the main patch has been applied. That would make the changes clearer.
--
David Jarvie.
KAlarm author & maintainer.
http://www.astrojar.org.uk/kalarm
More information about the kde-core-devel
mailing list