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 _______________________________________________ Freetype mailing list [email protected] https://lists.nongnu.org/mailman/listinfo/freetype
