Re: [ft] [ft-devel] Cleaning up ttinterp

[Werner LEMBERG]

Users who change

  TT_CONFIG_OPTION_INTERPRETER_SWITCH

and/or

  TT_CONFIG_OPTION_STATIC_INTERPRETER

(the latter only exists in file `ttinterp.c') please read!

> I was checking it out today and noticed that there are two main
> loops: default big switch, and non-default function-dispatch array.
> There was also two modes for the execution context: default MT-safe,
> non-default MT-unsafe.
>
> I removed the non-default codepaths.  And did some removing macros.
> I've put the results here so far:
>
>   https://github.com/behdad/freetype/tree/ttinterp-cleanup
>
> The first two commits remove about 1600 lines combined.

Nice!

> There's more cleanup that can be done, but I figured I shared this
> first and only continue if there's interest merging it.

Looks good to me, and yes, this seems to be the right thing.  However,
I just wonder whether there is a user who actually uses the different
code path...

So let's make this e-mail a poll.  If I don't get a response within a
week, I would like to incorporate your changes.


    Werner