> Looks good to me. One other tip is that you can fake a "continue"
> statement with "goto", which will let you reduce the amount of
> nested "if"s: [...]
Thanks, I will remember this for future code. For my snippet, though,
I think the current solution is legible enough.
> Another suggestion is to use a more descriptive name for your
> "luaotfload.patch_font" callback, since "patch-fonts" isn't the most
> informative. "add-to-mark" would probably be a good name.
Yeah, I've missed that. Will change locally.
Werner
