[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [ft-devel] ftdump functionality in ftinstpect
|
From: |
Werner LEMBERG |
|
Subject: |
Re: [ft-devel] ftdump functionality in ftinstpect |
|
Date: |
Mon, 08 Oct 2018 08:33:12 +0200 (CEST) |
> 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