>>> "AE" == Arash Esbati <ar...@gnu.org> writes:

> Hi Uwe,
> Uwe Brauer <o...@mat.ucm.es> writes:

> Thanks for the changeset.  I have some comments below:


> I think it makes sense to declare this as a verbatim macro (see below.)


Ok 
>> +    '("keywords" 1)
>> +    '("subjclass" ["Year"] "List of subjects")
>> +    '("thanks" 1)      

> This is already available in latex.el, so it can be removed here.


Subjclass is *not* defined defined in latex, I am confused here

>> +    '("title" ["Short Title"] "Title")
>> +    '("urladdr" 1))

> I think it makes sense to declare this as a verbatim macro, we insert:

> (add-to-list 'LaTeX-verbatim-macros-with-braces-local "email")
> (add-to-list 'LaTeX-verbatim-macros-with-braces-local "urladdr")

>> +   (TeX-run-style-hooks "amsmath" "amsthm")
>> +   (LaTeX-add-environments "abstract")
>> +   (when (and (featurep 'font-latex)
>> +              (eq TeX-install-font-lock 'font-latex-setup))
>> +     (font-latex-add-keywords '(("author" "[{")
>> +                                ("contrib" "[{")
>> +                                ("curraddr" "{")
>> +                                ("email" "{")
>                                    ("email" "")
> Remove the brace since is done by syntactic fontification.

For curraddr etc as well
>> +                                ("dedicatory" "{")
>> +                                ("keywords" "{")
>> +                                ("subjclass" "[{")
>> +                                ("title" "[{")
>> +                                ("thanks" "{")

> Already in font-latex.el

Subjclass is also not in font-latex.el, are you using top-quoting here?

I am confused

Uwe 

Attachment: smime.p7s
Description: S/MIME cryptographic signature

Reply via email to