Florent Rougon <[email protected]> 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 _______________________________________________ bug-auctex mailing list [email protected] https://lists.gnu.org/mailman/listinfo/bug-auctex
