Ticket #39710

microtype patch is wrong

Open Date: 2019-10-29 09:08 Last Update: 2019-11-18 07:13

Reporter:
(Anonymous)
Owner:
(None)
Type:
Status:
Closed
Component:
(None)
MileStone:
(None)
Priority:
5 - Medium
Severity:
5 - Medium
Resolution:
None
File:
None

Details

Hi,

in the latest patch to microtype, you've introduced a bug that results in exponentially slowing down compilation

\def\MT@register@font{%

\xdef\MT@font@list{\expandonce{\MT@font@list\MT@font},}% !!!

}

(Likewise for the other commands. This change is in https://de.osdn.net/projects/luatex-ja/scm/git/luatexja/commits/26142dfa8cafe366a32dff1459c8f0f4c60c83d3)

This will only add the command \MT@font to the list, not the actual font name (e.g. \T1/cmr/m/n/10), so that microtype will redo its font setup each and every time a font is loaded. This has been discovered by a user here: https://chat.stackexchange.com/transcript/message/52330126#52330126

I don't know why you need to patch these commands at all (the user's test document works fine for me if I just comment the patches), but in any case this is wrong. Actually, I would also prefer to do any patches required in microtype itself, since for example these commands happen to change in the next version.

You can reach me at w.m.l at gmx.net to discuss this,

best,

Robert (microtype maintainer)

Ticket History (3/3 Histories)

2019-10-29 09:08 Updated by: None
  • New Ticket "microtype patch is wrong" created
2019-11-08 15:38 Updated by: h7k
Comment

I wrote a new patch (commit) which uses \patchcmd.

2019-11-18 07:13 Updated by: h7k
  • Status Update from Open to Closed
Comment

I uploaded a new version (20191117.0) to CTAN.

Attachment File List

No attachments

Edit

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Login