On Mon, Oct 19, 2015 at 12:16:58PM +0200, Jean-Marc Lasgouttes wrote:
> Le 15/10/15 18:07, Richard Heck a écrit :
> >>The only changed behavior is that the CoordCache asserts when
> >>requiring a non existing cache entry, where insetDimension would
> >>return a dummy value. This can lead to some crashes, but I would say
> >>that these are welcome in some cases.
> >
> >Is there something sensible to do in release mode in this case? rather
> >than crash?
> 
> I updated the patch to return null values and now I wonder whether this was
> a good idea. Currently it throws a buffer exception and saves the file
> before crashing.
> 
> We could maybe keep this for 2.2alpha and change our mind if it causes too
> many problems. I do not think that relying on default values is a good idea.

Sounds like a good plan.

Scott

Reply via email to