Re: Split glyph contours in up/down segments for skylines (issue 569700043 by address@hidden)

[jonas . hahnfeld]

On 2020/05/08 14:24:18, dak wrote:
> On 2020/05/08 14:14:47, hahnjo wrote:
> 
> > assert(false) that none of the previous cases was true? That should
detect a
> > breaking change in FT_Outline pretty quickly.
> 
> else if (condition)
> { ... }
> else assert(false);
> 
> seems to make the assertion output comparatively useless.  Wouldn't it
be better
> to do such things as
> 
> else {
>   assert(condition);
>   ...
> }
> 
> instead?

I'm used to the following idiom from LLVM:
assert(0 && "Unknown outline tag");

https://codereview.appspot.com/569700043/