[tex-live] Extended page grouping patch makeindex in texlive 2011?

[Pander]

On 2011-07-28 10:45, Peter Breitenlohner wrote:
> On Wed, 27 Jul 2011, Pander wrote:
> 
>> Here is a newer patch against TeX Live 2011.
> 
> looking at this patch I noticed:

Thank you for looking into it and providing feedback

> (1) the changes for makeindex.1 mentioned in ChangeLog are missing

fixed

> (2) the patch for mkind.h is reversed.  Moreover
>   "Usage: %s [-ilqrcgBES] ..."
> indicates three options `-B', `-E', and `-S' without params. That ought to
> be something like
>   "Usage: %s [-ilqrcg] [-B bgr] [-E egr] [-S sgr] ..."

fixed and put these options more near the end

> (3) mkind.c
>   if ((bgr != -1 || egr != -1) && egr < bgr)
> is equivalent to
>   if (egr < bgr)
> 
> However, I'd suggest
>   if (egr != -1 && egr < bgr)

fixed

> with
>   bgr > -1 and egr == -1
> interpreted as egr == infinity
> 
> (4) would it be useful to require that (egr-bgr) is a multiple of sgr?

nope, that is up to the user

I also added the test case, see attached patch.

Please review the patch again.

Best regards,

Pander

> Regards
> Peter Breitenlohner <peb at mppmu.mpg.de>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: makeindexk-20110830.patch
Type: text/x-patch
Size: 7496 bytes
Desc: not available
URL: <http://tug.org/pipermail/tex-live/attachments/20110830/65bec351/attachment.bin>