[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Clean up and fix glyph contour generation nits. (issue 566080043 by
From: |
jonas . hahnfeld |
Subject: |
Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden) |
Date: |
Sun, 10 May 2020 03:03:57 -0700 |
On 2020/05/10 10:02:27, hanwenn wrote:
> On 2020/05/10 10:00:54, hahnjo wrote:
> > On 2020/05/10 09:29:48, hanwenn wrote:
>
https://codereview.appspot.com/566080043/diff/560020046/lily/freetype.cc#newcode149
> > > lily/freetype.cc:149: return ((Path_interpreter *) user)->moveto
(*to);
> > > On 2020/05/10 09:16:59, hahnjo wrote:
> > > > Do you want to create copies of the arguments? (here and for the
other
> > > > functions)
> > >
> > > yes. I find it easier to reason about and read. It's all inlined
anyway.
> >
> > const refs should work the same AFAICS
>
> Yes, I know. Do you want me to change this?
Yes, const refs avoid relying on compiler optimizations to remove the
copies.
https://codereview.appspot.com/566080043/
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), (continued)
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), jonas . hahnfeld, 2020/05/09
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), hanwenn, 2020/05/09
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), hanwenn, 2020/05/09
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), lemzwerg, 2020/05/10
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), jonas . hahnfeld, 2020/05/10
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), jonas . hahnfeld, 2020/05/10
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), hanwenn, 2020/05/10
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), jonas . hahnfeld, 2020/05/10
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), hanwenn, 2020/05/10
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden),
jonas . hahnfeld <=
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), hanwenn, 2020/05/10
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), hanwenn, 2020/05/10
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), lemzwerg, 2020/05/10