[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: assert in page breaking
From: |
Han-Wen Nienhuys |
Subject: |
Re: assert in page breaking |
Date: |
Wed, 24 Jan 2007 13:07:16 +0100 |
User-agent: |
Thunderbird 1.5.0.9 (X11/20061219) |
Joe Neeman escreveu:
> On 1/24/07, Han-Wen Nienhuys <address@hidden> wrote:
>> + if (chunks.size () != div.size () + 1)
>> + {
>> + programming_error ("did not find a valid page breaking
>> configuration");
>> + ignore_div = true;
>> + assert (0);
>>
>>
>> this is better, but can we skip the assert()?
>
> If this assert fails, it's a bug in the page breaker. I personally
> prefer the assert because it means that if the bug appears during
> "make web," I will find it instead of it being lost in the output.
> This check really shouldn't depend on the validity of the user input
> -- there are plenty of other sanity checks along the way to test for
> that.
It will show up on 'make check', because it would cause a diff in the
logfile.
>> The problem with assert is that it prevents people from getting a PDF
>> and looking at their score to see what the problem with it is.
>
> But the user builds are done with NDEBUG, right?
Yes, but does it degrade gracefully when the assert is not triggered?
--
Han-Wen Nienhuys - address@hidden - http://www.xs4all.nl/~hanwen
LilyPond Software Design
-- Code for Music Notation
http://www.lilypond-design.com