> 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

_______________________________________________
Freetype-devel mailing list
Freetype-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/freetype-devel

Reply via email to