GNU bug report logs - #21259
Regex of (this)pagestyle in TeX-complete-list

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 21259 in the body.
You can then email your comments to 21259 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Arash Esbati <esbati <at> gmx.de>
To: bug-auctex <at> gnu.org
Subject: Regex of (this)pagestyle in TeX-complete-list
Date: Fri, 14 Aug 2015 21:32:04 +0200

[Message part 1 (text/plain, inline)]

Hi all,

if I start emacs with `emacs --eval '(setq debug-on-error t)' &' type
the following a tex file:

    \thispagestyle{`C-M-i'

I get this error:

--8<---------------cut here---------------start------------->8---
Debugger entered--Lisp error: (error "Can't find completion for \"\\thispagestyle{\"")
  signal(error ("Can't find completion for \"\\thispagestyle{\""))
  error("Can't find completion for \"%s\"" "\\thispagestyle{")
  TeX-complete-symbol()
  call-interactively(TeX-complete-symbol nil nil)
  command-execute(TeX-complete-symbol)
--8<---------------cut here---------------end--------------->8---

It is due to a bug in the definition of `TeX-complete-list' in
`latex.el' (line 5913):

--8<---------------cut here---------------start------------->8---
(setq TeX-complete-list
      (append '(;; snipped
		("\\\\\\(this\\)?pagestyle{\\([A-Za-z]*\\)"
		 1 LaTeX-pagestyle-list "}"))
		TeX-complete-list))
--8<---------------cut here---------------end--------------->8---

The number refers to the wrong subgroup in the regex.  There are two
ways to fix this: Make `\\(this\\)' a shy-group or replace `1' with `2'.

Please find attached a patch going the second route since it looks more
coherent to the rest of definitions in `TeX-complete-list'.  

As always, any comments welcome.

Best, Arash

[0001-Fix-the-subgroup-number-in-a-regex.patch (text/x-patch, attachment)]

Message #10 received at 21259-done <at> debbugs.gnu.org (full text, mbox):

From: Mosè Giordano <mose <at> gnu.org>
To: Arash Esbati <esbati <at> gmx.de>
Cc: 21259-done <at> debbugs.gnu.org
Subject: Re: bug#21259: Regex of (this)pagestyle in TeX-complete-list
Date: Sat, 15 Aug 2015 15:19:22 +0200

Hi Arash,

2015-08-14 21:32 GMT+02:00 Arash Esbati <esbati <at> gmx.de>:
> Hi all,
>
> if I start emacs with `emacs --eval '(setq debug-on-error t)' &' type
> the following a tex file:
>
>     \thispagestyle{`C-M-i'
>
> I get this error:
>
> --8<---------------cut here---------------start------------->8---
> Debugger entered--Lisp error: (error "Can't find completion for \"\\thispagestyle{\"")
>   signal(error ("Can't find completion for \"\\thispagestyle{\""))
>   error("Can't find completion for \"%s\"" "\\thispagestyle{")
>   TeX-complete-symbol()
>   call-interactively(TeX-complete-symbol nil nil)
>   command-execute(TeX-complete-symbol)
> --8<---------------cut here---------------end--------------->8---
>
> It is due to a bug in the definition of `TeX-complete-list' in
> `latex.el' (line 5913):
>
> --8<---------------cut here---------------start------------->8---
> (setq TeX-complete-list
>       (append '(;; snipped
>                 ("\\\\\\(this\\)?pagestyle{\\([A-Za-z]*\\)"
>                  1 LaTeX-pagestyle-list "}"))
>                 TeX-complete-list))
> --8<---------------cut here---------------end--------------->8---
>
> The number refers to the wrong subgroup in the regex.  There are two
> ways to fix this: Make `\\(this\\)' a shy-group or replace `1' with `2'.
>
> Please find attached a patch going the second route since it looks more
> coherent to the rest of definitions in `TeX-complete-list'.

Agreed.  Patch installed, thanks!

Bye,
Mosè

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.