Re: patch 1

[Tassilo Horn]

Uwe Brauer <oub@mat.ucm.es> writes:

Hi Uwe,

> -        style/tex-live.el
> +        style/tex-live.el  style/algorithm.el \
                                                 ^
Like in the other patch, no need for the \.

> +;; Copyright (C) 2016--2020 Free Software Foundation, Inc.
                    ^^^^^^^^^^
I think the year of adding it publicly is what's important here, not
the years you've maintained it privately.

> +(declare-function font-latex-add-keywords
> +               "font-latex"
> +               (keywords class))

You don't use it, so don't forward-declare it.

> +(TeX-add-style-hook
> + "algorithm"
> + (lambda ()
> +   (TeX-run-style-hooks "article")

Why is that needed?

> +   (LaTeX-add-environments
> +    '("algorithm"  LaTeX-env-algorithm ))
> +   (TeX-add-symbols
> +    '("listofalgorithms" 0))
> +   (add-to-list 'LaTeX-label-alist '("algorithm" . LaTeX-algorithm-label) t)
                                                      ^^^^^^^^^^^^^^^^^^^^^
That's not defined in the style...

Bye,
Tassilo