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

Reply via email to