Re: [PATCH] New margin handling - final version (updated)

[Patrick McCarty]

On Fri, Sep 11, 2009 at 3:33 PM, Michael Käppler <address@hidden> wrote:
> Hi Neil,
> I'm fine with all your comments...
>>
>> The regression test would probably benefit from being split into
>> several separate tests, particularly the final check-consistency
>> setting, which can then have #(ly:set-option 'warning-as-error #f)
>> added to it in anticipation of the mythical time when we can switch
>> this option on for regression testing. :)
>
> ...however, I don't really understand the benefit of splitting the regtest.
> Do you propose to have separate files for each combination of settings?
>
> And if warning-as-error was set to true, the regtest would fail. I don't
> think that is intended, since the >absence< of the warning would be wrong
> behaviour, not the appearance.

If you *want* the warning(s) for regtests, add

  #(ly:set-option 'warning-as-error #f)

as Neil suggested.  Eventually, this option will be set to #t for
regtest compilation, thus the need for overriding the default
behavior.

Thanks,
Patrick