search

alexeyr / company-auctex

company-mode autocompletion for auctex
49 stars 18 forks source link

Fix for templates without arguments and some cleanup. #6

Closed pronobis closed 9 years ago

pronobis commented 9 years ago

company-auctex generated unnecessary braces for environments without arguments in the company-auctex-arg-lookup-table list. This resulted e.g. in

\begin{itemize}{|}
\end{itemize}

57e25db fixes that.

Then, I noticed that the argument number is not used at all in the snipped, so I cleaned up the code in 9b8b85f.

alexeyr commented 9 years ago

Looks good to me. I assume you tested it? (Unfortunately, I don't have Emacs set up at the moment.)

pronobis commented 9 years ago

Seems to work well for me.

pronobis commented 9 years ago

I added one more commit adding lengths to macros. I can separate it to another PR if you want.

alexeyr commented 9 years ago

No need, thank you.