On 08/05/2014 17:00, Ulrike Fischer wrote:


> I tested a bit with various files and it seems ok. But -- as the
> question on tex.sx shows -- the bug didn't always lead to errors but
> could simply produce faulty output and it is not so easy to check if
> somewhere an "and" or a comma is missing

Thanks.

 With this change reverted the only remaining change is adding \long everywhere
so there shouldn't be any change other than things that were error before are now allowed.
(Although code that is explicitly checking a definition with \patchcmd or
\checkcommand etc can of course fail if _any_ change is made such failures are usually pretty obvious though.



> If one loads the new xkeyval v2.6c before keyval the error is still
> there. So xkeyval needs a patch too.

Yep it's an identical patch needed (in xkeyval's keyval.tex and xkeyval.tex)
Sorry to Hendri Adriaens (I gave him a prerelease of keyval and suggested he update to
match) I was just holding off until I have confirmed this is OK before suggesting xkeyval is changed again.

David



$ diff /usr/local/texlive/2014/texmf-dist/tex/generic/xkeyval/keyval.tex .
48c48
< \long\def\KV@@sp@b#1##1 \@nil{\KV@@sp@c\@empty##1}%
---
> \long\def\KV@@sp@b#1##1 \@nil{\KV@@sp@c##1}%
51c51
< \long\def\KV@@sp@c#1\@nil#2\relax#3{\XKV@toks\expandafter{#1}\edef#3{\the\XKV@toks}}
---
> \long\def\KV@@sp@c#1\@nil#2\relax#3{\XKV@toks{#1}\edef#3{\the\XKV@toks}}


$ diff /usr/local/texlive/2014/texmf-dist/tex/generic/xkeyval/xkeyval.tex .
786c786
<   \XKV@r@placepointers\@empty#1\usevalue\@nil
---
>   \XKV@r@placepointers#1\usevalue\@nil
789c789
<   \expandafter\XKV@addtomacro@n\expandafter\XKV@tempa\expandafter{#1}%
---
>   \XKV@addtomacro@n\XKV@tempa{#1}%





________________________________________________________________________
The Numerical Algorithms Group Ltd is a company registered in England
and Wales with company number 1249803. The registered office is:
Wilkinson House, Jordan Hill Road, Oxford OX2 8DR, United Kingdom.

This e-mail has been scanned for all viruses by Star. The service is
powered by MessageLabs.
________________________________________________________________________