freetype-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [ft-devel] Experimental: v38 interpreter with minimal backwards comp


From: Werner LEMBERG
Subject: Re: [ft-devel] Experimental: v38 interpreter with minimal backwards compatibility mode and linear advance widths
Date: Thu, 17 Mar 2016 09:25:02 +0100 (CET)

> Another one.

Thanks.

There are a lot of repetitive texts in your patch, like

  /* Given ClearType's virtual increase of resolution on the X-axis, rounding
   * to the physical pixel grid there doesn't make much sense. Native ClearType
   * fonts know this, the rest (where backwards compatibility applies) may or
   * may not. Short out rounding if freedom vector is parallel to X-axis. */

This is hard to main in case there should be ever something to change.
Can you reduce this amount of duplication?  For example, introduce a
comment section with a numbered list of remarks near the top of the
file, then refer to it in the code.


    Werner



reply via email to

[Prev in Thread] Current Thread [Next in Thread]