On Fri, May 8, 2020 at 4:14 PM <jonas.hahnf...@gmail.com> wrote: > > On 2020/05/08 14:03:23, hanwenn wrote: > > > https://codereview.appspot.com/569700043/diff/582060043/lily/freetype.cc#newcode162 > > lily/freetype.cc:162: } > > On 2020/05/08 08:15:59, hahnjo wrote: > > > This code looks very familiar. Don't we already have it in > > > make_draw_bezier_boxes? > > > > yes. > > So can we reuse that?
No, because there the segments don't have orientation. I also want to use integer math at some point here, so we'd have extra work to disentangle the code again. > > sure. But what would we assert? FT reserves the right to put other > things in tag > > bits. > > assert(false) that none of the previous cases was true? That should > detect a breaking change in FT_Outline pretty quickly. I'm going to defer to Werner for advice here. Given that it was like this before, I think it's better it as is in this change. -- Han-Wen Nienhuys - hanw...@gmail.com - http://www.xs4all.nl/~hanwen