各ビルダのtableメソッドのリファクタリング

[kmuto]

#1330 の準備の一環として、tableメソッドの整理をしました。

• sepidx解析部分はほとんどのビルダで共通なので、Builder#parse_table_rowsを導入
• その後のtrを各処理する部分もほぼ共通なので、Builder#table_trを導入

ただ、latexbuilderやidgxmlなどでどうしても渡したい固有の変数があり、仕方がないのでインスタンス変数を使って渡すようにしているところもあります。

[takahashim]

基本的な方針としては良さそうですが、細かいところでいくつか気になるところが。

• parse_table_rows() にまとめるのは良さそうです。
• table_tr()は名前がわかりにくい（<tr>を生成するやつかと思ったら違った…）ので、table_rows()でどうでしょうか。
• ReVIEW::Bulder#table は、汎用化できるなら汎用化して必要ないところではサブクラスで再定義しないようにするか、汎用化できなさそうならNotImplementedErrorにした方がよいかと思います。LaTeX、Markdown、Plaintextあたりは汎用化できそう？

[kmuto]

ありがとうございます、Builder#tableはそういえばせっかく共通化作業をしたのにサブクラスから消すの忘れてました。

[kmuto]

これでどうでしょう。

[takahashim]

ありがとうございます！
まずはこれで行ってみますか。