Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-22 Thread Arash Esbati
Hi Mosè, Mosè Giordano writes: > 2016-12-20 12:19 GMT+01:00 Arash Esbati : >> The in and out file are attached. Running make in tests directory >> shows that my test fails due to this excerpt: > > I think that now it'll work ;-) Adding > > (setq

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-20 Thread Mosè Giordano
Hi Arash, 2016-12-20 12:19 GMT+01:00 Arash Esbati : > The in and out file are attached. Running make in tests directory > shows that my test fails due to this excerpt: I think that now it'll work ;-) Adding (setq TeX-parse-self t) before (LaTeX-mode) should

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-20 Thread Arash Esbati
Hi Mosè, Mosè Giordano writes: > first of all, thanks for your work! If you believe this change is > worth a mention in changes.texi, drop there a few lines. Thanks, I could put some lines into changes.texi. > 2016-12-16 15:57 GMT+01:00 Arash Esbati :

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-20 Thread Mosè Giordano
Hi Arash and Ikumi, first of all, thanks for your work! If you believe this change is worth a mention in changes.texi, drop there a few lines. 2016-12-16 15:57 GMT+01:00 Arash Esbati : > Do you have a hint how I can run style hooks > (e.g. tabularx.el) while running

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-19 Thread Ikumi Keita
Hi Arash and all, > Following up myself, I did some more testing with your code and it > worked like a charm and since there was no objections, I pushed the code > incl. a test case and an update to the docs to git. I added a style for > dcolumn.sty and also fixed a bug in array.el. > Again,

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-19 Thread Arash Esbati
Hi Ikumi and all, Arash Esbati writes: > Ikumi Keita writes: >> >> Apparently recursive call of LaTeX-array-count-columns is needed do the >> task. How about this one? >> [...] > > Thanks! I'm fine with your suggestions. I have already pushed

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-17 Thread Arash Esbati
Hi Ikumi, Ikumi Keita writes: >> 2 thing occurred to me when hitting `M-RET' in tabular environments: >> Current code in `LaTeX-array-count-columns' cannot handle multi column >> specs like `*{num}{spec}' and optional args to column specs S and s from >> siunitx.el. I have

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-16 Thread Ikumi Keita
Hi Arash and all, > 2 thing occurred to me when hitting `M-RET' in tabular environments: > Current code in `LaTeX-array-count-columns' cannot handle multi column > specs like `*{num}{spec}' and optional args to column specs S and s from > siunitx.el. I have a patch to handle both issues. Can

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-15 Thread Mosè Giordano
Hi Arash, 2016-12-14 15:52 GMT+01:00 Arash Esbati : > Hi Mosè, > > Mosè Giordano writes: > Thanks for testing! I can take the file I posted with some other > environments and add it as `tabular-ampersand-count.tex' under > auctex.git/tests/latex. Is

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-15 Thread Ikumi Keita
Hi Arash, > Thanks for your response. This syntax is actually standard LaTeX, e.g.: > \documentclass{article} > \begin{document} > \_ > \begin{tabular}[t]{@{}*{2}{l}r} > 1 & 2 & some longish text \\ > 1 & 2 & some text \\ \hline > \end{tabular} > \_

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-14 Thread Arash Esbati
Hi Ikumi, Ikumi Keita writes: >> 2 thing occurred to me when hitting `M-RET' in tabular environments: >> Current code in `LaTeX-array-count-columns' cannot handle multi column >> specs like `*{num}{spec}' and optional args to column specs S and s from >> siunitx.el. I have

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-14 Thread Ikumi Keita
Hi Arash, > 2 thing occurred to me when hitting `M-RET' in tabular environments: > Current code in `LaTeX-array-count-columns' cannot handle multi column > specs like `*{num}{spec}' and optional args to column specs S and s from > siunitx.el. I have a patch to handle both issues. Instead of

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-14 Thread Arash Esbati
Hi Mosè, Mosè Giordano writes: > 2016-12-14 12:34 GMT+01:00 Arash Esbati : >> >> 2 thing occurred to me when hitting `M-RET' in tabular environments: >> Current code in `LaTeX-array-count-columns' cannot handle multi column >> specs like `*{num}{spec}'

Re: [AUCTeX-devel] Extending `LaTeX-array-count-columns'

2016-12-14 Thread Mosè Giordano
Hi Arash, 2016-12-14 12:34 GMT+01:00 Arash Esbati : > Hi all, > > 2 thing occurred to me when hitting `M-RET' in tabular environments: > Current code in `LaTeX-array-count-columns' cannot handle multi column > specs like `*{num}{spec}' and optional args to column specs