> This patch adds the basic functionality of ftdump in ftinspect.
> More changes/features will be added in subsequent patches.

Very nice!  Do you plan to work more on ftinspect?  This would be
great!

> Please have a look & suggest the required changes for refinement of
> this patch.

Some comments.

* I think it would be best if all the information is present in a
  single window.  If necessary, there could be tabs to split the
  contents into smaller chunks.

* Don't insert a space before the double colon character (`:') – this
  is correct for French typography but not for English.

* For the charmap info it would be nice if the meaning of the numbers
  could be shown also, e.g. PID 3 = MS Windows, EID 1 = Unicode BMP,
  etc., etc.


    Werner
_______________________________________________
Freetype-devel mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/freetype-devel

Reply via email to