Re: [PATCH v2] Fix crash when a stencil routine is missing

[Graham Percival]

On Sun, Jul 05, 2009 at 06:01:21PM -0700, Patrick McCarty wrote:
> On Sun, Jul 05, 2009 at 05:39:45PM -0700, Graham Percival wrote:
> > On Sun, Jul 05, 2009 at 02:29:44PM -0700, Patrick McCarty wrote:
> > > 1.) I used Jan's suggestion for -d option name, but I'm not sure this
> > > is the best name.  What should the option be called?
> > 
> > -dwarning-as-error
> > looks fine to me.
> 
> Okay.  On second thought, it looks decent to me too.

Wasn't that your initial proposal?  Did I misread the patch?

> > ... oh wait, here it is:
> > #(ly:set-option 'point-and-click #f)
> 
> I think Han-Wen suggested to enable this option for the entire
> regression test suite, so adding this option to every single
> regression test would not be reasonable.
> 
> In other words, some makefile hacking is needed, which I would
> probably need to delegate.

I doubt that the regtests compile without warnings, and I'm not
certain that we have enough Frogs to eat all the issues that would
be added if we went this route.

(I'd love to be proven wrong about this, though!)

Cheers,
- Graham