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/
