2007/7/24, Martin Aumueller <<a href="mailto:aumuell@reserv.at">aumuell@reserv.at</a>>:<div><span class="gmail_quote"></span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Sunday 22 July 2007, Bart Cerneels wrote:<br>> SVN commit 690980 by shanachie:<br>><br>> Some improvements.<br>><br>> Note: the Q_ASSERTS are here for debugging. I'll remove them when the code<br>> is stable.
<br><br>IMHO, no need for removing them: it's a good way of documenting pre- and<br>post-conditions. And probably the code is wrong anyway in some place if the<br>assertion fails. And if the code is stable, it should get compiled
<br>with -DQT_NO_DEBUG, and then Q_ASSERT becomes a no-op.</blockquote><div><br>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.<br>
But I agree there are some that are totally unnecessary and they will be removed. </div><br><br>Greetings</div>