googlefonts / roboto-classic

Development of a Roboto Variable font
SIL Open Font License 1.1
154 stars 15 forks source link

Optimizing preprogram #39

Closed dberlow closed 4 years ago

dberlow commented 4 years ago

Mike I noticed while I was separating out the XML of the hints that there are an awful lot of function definitions in the pre-program. I’m wondering if we can easily establish which ones are required by searching the text of the glyph hints, or if you know what’s actual being used, let me know if that’s worth optimizing.

Thanks.

mikedug commented 4 years ago

David, I would be very nervous of reducing the function definitions, as I do not have a clear idea, which are needed. Unless this is a big file size saving, and something that is absolutely required, I would recommend leaving as is. thanks mike

dberlow commented 4 years ago

Hey Mike, I don’t suggest this calmly, or in any hurry. I’m just interested in the possibilities as we gather up ways to optimize file size.

mikedug commented 4 years ago

sure, I can have a look at this.

I get nervous about removing FN Definitions, the Autohinter is a beast, many functions, I believe rely on or somehow reference others.

we can chat about this as we go along.

mike

On 8 Jan 2020, at 14:40, David Berlow notifications@github.com wrote:

Hey Mike, I don’t suggest this calmly, or in any hurry. I’m just interested in the possibilities as we gather up ways to optimize file size.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/TypeNetwork/Roboto/issues/39?email_source=notifications&email_token=ANKNGONHV6W6XURDCATHZ2LQ4XQUBA5CNFSM4KEJDB62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIMWBNA#issuecomment-572088500, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANKNGOK3LDRYFCK3JDAUVHDQ4XQUBANCNFSM4KEJDB6Q.

davelab6 commented 4 years ago

This seems to be something that will need to be done in a general purpose, algorithmic way. However, my guess is that I don't want to prioritize this over more important projects, so closing for now.

dberlow commented 4 years ago

Agreed. Coincidentally, solving it was one of my last ideas yesterday. We’ll try it on the upcoming hints of extremo and if that works it’s likely to ttx copy/paste into other fonts that use vtt autohint.

On Apr 1, 2020, at 8:39 AM, Dave Crossland notifications@github.com wrote:

 This seems to be something that will need to be done in a general purpose, algorithmic way. However, my guess is that I don't want to prioritize this over more important projects, so closing for now.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.