Re: [PATCH] docs: Detect variable fonts and suggest removing them

[Randy Dunlap <rdunlap@infradead.org>]

>>> ---
>>> Cc: linux-doc@vger.kernel.org
>>> ---
>>>  Documentation/Makefile          |  2 +-
>>>  MAINTAINERS                     |  1 +
>>>  scripts/check-variable-fonts.sh | 30 ++++++++++++++++++++++++++++++
>>>  3 files changed, 32 insertions(+), 1 deletion(-)
>>>  create mode 100755 scripts/check-variable-fonts.sh
>>>
>>
>> I don't have much to say about the patch except that it seems like a good idea...
>>
>> However, some of the terminology could be improved IMO.
>> E.g., "variable type" or "variable font" should be something like
>> variable-width font or proportionally-spaced font.
> 
> Good point.
> 
> I need to be more careful in using terms rarely found in kernel development.
> 
> In this case, "variable font" means "font file which employs the OpenType
> font variations technology".
> 
> There is a Wikipedia page at: https://en.wikipedia.org/wiki/Variable_font
> 

Ah, I see. Thank you.

>>
>> "static ones" should be something like fixed-space fonts or monospaced fonts.
>>
>> Unless I just completely don't understand the uses of "variable" and "static" here.
> 
> "static" here means "non-variable".
> 
> I'll expand the changelog a bit in v2 with your comments in mind.
> 
>         Thanks, Akira

-- 
#Randy