QDateTime and kdbgstream

Stephan Kulow coolo at kde.org
Tue Nov 12 12:32:58 GMT 2002


Am Tuesday 12 November 2002 12:54 schrieb Holger Freyther:
> On Tuesday 12 November 2002 12:01, David Faure wrote:
> > On Tuesday 12 November 2002 11:47, Holger Freyther wrote:
> > > Hi devels,
> > > I would like to commit the following patch after KDE3.1 was released.
> > > It adds 3 operator<<'s. One for QDateTime, one for QTime and one for
> > > QDate.
> >
> > Looks good. This makes me wonder why kdebugclasses.h is all inlined.
>
> I do not really see a reason for that too.
>
> > I mean, with your approach, only forward declarations are needed in the
> > .h file, so it's indeed no problem to add some more types to kdebug.h
> > With kdebugclasses.h, the calling code has to include all those
> > additional headers. Why don't we use this approach (non-inline methods)
> > for all the types handled by kdebugclasses.h? Then we could move all
> > those to kdebug.h, since it would involve no #include cost. (Who cares
> > about runtime costs, for a kdDebug statement?)
>
> For khtml or kio http this could be a problem ( speed issue ) but I think
> not that either khtml or kio http is using one of the kdebugclasses
> operators.
>
> > Ah BTW your patch misses empty inline versions for kndbgstream (the
> > --disable-debug case).
>
> Ok the following patch also covers that case.

Then please prepare the pratical death of kdebugclasses.h with your patch
and keep it for after 3.1 :)

Greetings, Stephan





More information about the kde-core-devel mailing list