extragear/multimedia/amarok/src/podcasts

[Bart Cerneels]

2007/7/24, Martin Aumueller <aumuell at reserv.at>:
>
> On Sunday 22 July 2007, Bart Cerneels wrote:
> > SVN commit 690980 by shanachie:
> >
> > Some improvements.
> >
> > Note: the Q_ASSERTS are here for debugging. I'll remove them when the
> code
> > is stable.
>
> IMHO, no need for removing them: it's a good way of documenting pre- and
> post-conditions. And probably the code is wrong anyway in some place if
> the
> assertion fails. And if the code is stable, it should get compiled
> with -DQT_NO_DEBUG, and then Q_ASSERT becomes a no-op.


You are right Martin, the asserts are just a development and debug tool.
Specifically calling  the read*() functions at the wrong time would be very
hard to debug without Q_ASSERT.
But I agree there are some that are totally unnecessary and they will be
removed.


Greetings
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20070724/88b794d5/attachment.html