[tex-eplain] Expanding \indexnameseparator (was: optional arguments)

[Oleg Katsitadze]

On Mon, Mar 23, 2009 at 02:31:46PM -0400, John Culleton wrote:
> The solution would be to modify eplain such that instead of 
> inserting  \indexnameseparator the \indexname macro would 
> simply insert a comma. This is the default choice anyhow 
> and the one that conforms to standard indexing practice. 

Well, the whole point of \indexnameseparator is to provide
flexibility, in case someone wants something different from the
standard indexing practice.  But I agree it would be nice to have ", "
instead of "\indexnameseparator" inside the .idx file.  So I suggest
the patch below.  It seems backwards-compatible, except in situations
described by John, but such documents are already broken anyway.

If there are no objections, I'll install the patch.

Thanks,
Oleg


Index: xeplain.tex
===================================================================
RCS file: /cvsroot/eplain/eplain/xeplain.tex,v
retrieving revision 1.47
diff -u -r1.47 xeplain.tex
--- xeplain.tex 20 Nov 2007 21:51:02 -0000  1.47
+++ xeplain.tex 24 Mar 2009 04:28:03 -0000
@@ -2842,7 +2842,7 @@
   \ifx\temp\empty
     \toks@ = {}%
   \else
-    \toks@ = {\idxnameseparator #1}%
+    \toks@ = \expandafter{\idxnameseparator #1}%
   \fi
   \toks2 = {#2}%
   %