Re: [AUCTeX-devel] Misc patches

[Mosè Giordano]

Hi Ikumi,

2016-04-17 8:20 GMT+02:00 Ikumi Keita <address@hidden>:
> Hi Mosè,
>
>> Thanks for your patches, I applied all of them but
>
>>     Temporarily bind `TeX-insert-macro-default-style' to 'show-optional-args'
>
>> If it is relevant only for description environment, why not binding
>> the variable to `show-optional-args' only when `LaTeX-environment' is
>> equal to "description"?  I understand the point of your patches, in
>> that environment the optional argument is virtually mandatory, but
>> there are other cases in which the item macro has really optional
>> arguments (for example when beamer package is used).
>
> If I understand correctly, `LaTeX-item-argument' is provided for
> environments where the optional argument is virtually mandatory.
> (Correct me if I'm wrong.)  The output of `grep LaTeX-item-argument *.el
> */*.el' is:
>
> latex.el:(defun LaTeX-item-argument ()
> latex.el:  (set (make-local-variable 'LaTeX-item-list) '(("description" . 
> LaTeX-item-argument)
> style/currvita.el:   (add-to-list 'LaTeX-item-list '("cvlist" . 
> LaTeX-item-argument))
> style/enumitem.el:      (add-to-list 'LaTeX-item-list `(,env . 
> LaTeX-item-argument)))))
> style/enumitem.el:     (add-to-list 'LaTeX-item-list '("description*" . 
> LaTeX-item-argument)))
> style/enumitem.el:           (add-to-list 'LaTeX-item-list `(,name . 
> LaTeX-item-argument)))
> style/paralist.el:   (add-to-list 'LaTeX-item-list '("compactdesc" . 
> LaTeX-item-argument))
> style/scrbase.el:(add-to-list 'LaTeX-item-list '("labeling" . 
> LaTeX-item-argument))
>
> The function is called only in environments that are added to
> `LaTeX-item-list' in the form (envname . LaTeX-item-argumet).  For other
> environments, `LaTeX-item-argument' does not interfere at all.

Yes, you're completely right.  I just installed that patch, thank you.

Bye,
Mosè