[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: |
Han-Wen Nienhuys |
Subject: |
Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden) |
Date: |
Sun, 10 May 2020 10:57:58 +0200 |
On Sun, May 10, 2020 at 10:38 AM <address@hidden> wrote:
>
> On 2020/05/09 20:38:12, hanwenn wrote:
> > it's separate because it causes formatting changes. I like to keep the
> bugfixes
> > and performance fixes separate. (Performance fixes should not cause
> regtest
> > differences)
>
> Sounds fair, but I don't understand the relation of the two changes:
> - Both touch the very same function, and with Patch Set 1 both shared
> half of the patch.
> - With the recent Patch Set, this looks more advanced than
> https://codereview.appspot.com/569700043. So does it replace the former?
> - If not, how are the two patches supposed to be ordered? This one
> first, or the other, or does one contain the changes of the other?
>
> Until this is clear, there is no point in reviewing either IMHO.
The bugfix comes first.
--
Han-Wen Nienhuys - address@hidden - http://www.xs4all.nl/~hanwen
- 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/09
- 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),
Han-Wen Nienhuys <=
- 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, 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), hanwenn, 2020/05/10
- Re: Clean up and fix glyph contour generation nits. (issue 566080043 by address@hidden), lemzwerg, 2020/05/10