Re: indentation problem II

[Arash Esbati]

Hi Keita,

Ikumi Keita <ikumi@ikumi.que.jp> writes:

>>>>>> Arash Esbati <arash@gnu.org> writes:
>> diff --git a/style/amsmath.el b/style/amsmath.el
>> index 5e7812e7..f72f8be8 100644
>> --- a/style/amsmath.el
>> +++ b/style/amsmath.el
>> @@ -57,7 +57,8 @@
>>      "align*" "gather*" "flalign*" "multline*" "equation*"
>>      "split"
>>      "cases"
>> -    "matrix" "smallmatrix" "pmatrix" "bmatrix" "Bmatrix" "vmatrix" "Vmatrix"
>> +    "matrix" "smallmatrix"
>> +    "bmatrix" "Bmatrix" "pmatrix" "vmatrix" "Vmatrix"
>>      "subequations"
>>      '("subarray" "Alignment"))
>
> This part can be omitted.

I only did that to reduce the line width which was about 80 or so.

>> +(defun LaTeX-item-equation-matrix ()
>
> Maybe the function name should be reconsidered since it has little
> to do with equation.

I admit that amsmath.el is not really consistent about name-space.
Other functions in that style also go with '-equation-'.  We could clean
up, sometimes :-]

>> +        (insert (make-string n ?&))))))
>
> Maybe
> (insert-char ?& n)
> is better. (Sorry, I wasn't aware of this usage of `insert-char' when I
> wrote the similar code for `LaTeX-insert-ampersands'.)

Thanks for the hint -- I wasn't aware of it as well.

> P.S. Oops, it's too late. You already have installed the commit in the
> git repo.

Thanks for your comments.  Do you think we should touch the style with
any of the above?

Best, Arash