Joseph Wright
Joseph Wright
You planning to work on this?
Suggestion here is we tidy up and pre-load the corrected data in the format?
Ah, I see what's happened - I streamlined the auxiliaries a bit, but I'd not allowed for the fact that there needs to be different handling for token lists and...
We also need to cover `fp`: I think might be best to use a low-level test here (`\token_if_macro:NTF`), as this really is about how things are implemented.
Also, there are some stray braces (likely going back a _long_ way - perhaps we had `o`-type expansion at one point). @muzimuzhi do you want to make a PR or...
> > We also need to cover `fp` > > Not necessary, as `\fp_set:Nn \l_tmpa_fp {\l_tmpa_fp }` is a valid assignment. But as `\token_if_macro:NTF` is faster than `\str_case_e:nnTF`, expanding `fp`...
I have a shorter approach I'll go with: PR very shortly
We had a bit of discussion in this direction in the team: I can't fine the issue it was in! Your suggestion is an interesting take: lets see what the...
Did one of your suggested improvements to `\ShowHooks` touch on this?
@jlaurens I think we had no issue with the core idea in #1250 - but it needs to be focussed just on that change.