Re: [Bug-AUCTeX] 11.87; Little problems with 'file' in TeX-command-expand/TeX-expand-list

[Tassilo Horn]

Florent Rougon <address@hidden> writes:

Hi Florent,

> However, it is still possible to trigger an endless loop with growing
> memory use in AUCTeX by trying to compile a file named %t.tex. After
> the first expansion, it becomes %t.tex.tex, then %t.tex.tex.tex,
> etc. The first attached patch
> (auctex_TeX-command-expand_filename_expansion_bug.patch) fixes this.

Thanks, good catch.  I've applied this patch.  However, it's quite on
the borderline to "not so tiny patch".  So would you please be willing
to sign a copyright assignment so that we can easily take your
contributions in the future?

> I also suggest applying a second patch on top of both
> auctex_TeX-expand-list.patch from my previous bug report (dated Sun,
> 24 Aug 2014 16:01:21 +0200) and
> auctex_TeX-command-expand_filename_expansion_bug.patch from this mail.
> This second patch
> (auctex_TeX-command-expand_readability_improvement.patch) would be
> useful IMHO because there are two different bindings for the name
> 'file' used in TeX-command-expand, one of which is used when expanding
> entries from TeX-expand-list (the bad one in the case of %' IMHO),
> which is rather confusing for someone reading the code.

That patch doesn't apply for me failing in the tex.el hunk.  And I'm not
sure if its a good idea to rename the function parameter.  If you'd
rename the let-bound `file', then you wouldn't need to change the
`TeX-expand-list' entry, right?

Bye,
Tassilo