micoloretan / selnolig

All files associated with the lualatex package "selnolig"
22 stars 6 forks source link

Keeplig #11

Closed uliska closed 5 years ago

uliska commented 5 years ago

Since I've for the first time looked into a package I'm literally using for every document (THANK YOU!) I had a few coding ideas, and this is one of them.

Wrapping multiple patterns in one \keeplig macro makes the code slightly more readable but above all will significantly reduce the number of times the TeX/Lua border has to be crossed with function calls.

The second commit applies the change to the beginning of the German patterns file - only a part because I didn't come up with a reasonable automatic solution, and because (of course) I have no idea whether the change will be accepted anyway.

This builds on top of the code presented in #10

uliska commented 5 years ago

Just saw that the "Files changed" diff shows all the changes including those of my previous merge commit. So in order to inspect this PR one should either first merge the other or only look at the two last commits.