On Wed, Jan 17, 2024 at 10:05:33AM +0100, Anthonin Bonnefoy wrote: > > I do realize the same is true for plain \bind, but it seems > > like a bug there too. > > The unscanned bind's parameters are discarded later in the > HandleSlashCmds functions. So adding the ignore_slash_options() for > inactive branches scans and discards them earlier. I will add it to > match what's done in the other commands but I don't think it's > testable as the behaviour is the same unless I miss something.
Hmm. So it does not lead to any user-visible changes, right? I can get your argument about being consistent in the code across the board for all the backslash commands, though. > I did add the \bind, \bindx, \close and \parse to the inactive branch > tests to complete the list. Could you split the bits for \bind into a separate patch, please? This requires a separate evaluation, especially if this had better be backpatched. -- Michael
signature.asc
Description: PGP signature