[debbugs-tracker] bug#21259: closed (Regex of (this)pagestyle in TeX-complete-list)

[GNU bug Tracking System]

Your message dated Sat, 15 Aug 2015 15:19:22 +0200
with message-id <address@hidden>
and subject line Re: bug#21259: Regex of (this)pagestyle in TeX-complete-list
has caused the debbugs.gnu.org bug report #21259,
regarding Regex of (this)pagestyle in TeX-complete-list
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
21259: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=21259
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: Regex of (this)pagestyle in TeX-complete-list Date: Fri, 14 Aug 2015 21:32:04 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5

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
Description: Text Data

--- End Message ---

> --- Begin Message ---
>
>
>
> 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 <address@hidden>:
> > 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è
>
>
> --- End Message ---