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/