Re: [AUCTeX-devel] [PATCH] A fix for longtable and a new style

[Mosè Giordano]

Hi Arash,

2015-09-27 10:24 GMT+02:00 Arash Esbati <address@hidden>:
> Hi all,
>
> please find attached 2 patches:
>
> * style/newfloat.el: New file.
>
> * style/longtable.el ("longtable"): Make package lengths available
> through `LaTeX-add-lengths' and not `TeX-add-symbol'.
>
> Any comments welcome.

Only a minor nitpicking: I think it's better to append entries to
`LaTeX-label-alist', because it's a customizable variable and if you
prepend elements to it you'll override possible custom settings.
`pushnew' can't append to variables, but my understanding is that it's
safer than `add-to-list' when it comes to let-bound variables (can
someone please confirm this?), but this isn't the case, so you could
safely use `add-to-list' with t `append' argument.

Bye,
Mosè