[Seasar-user:21666] Re: S2JDBC の AutoSelect への includes() と excludes() 実装

2013年 7月 14日 (日) 16:55:47 JST

ふたがわ (jfut) です。

On 2013/07/14 16:30, Koichi Kobayashi wrote:

> 乙です！
> API 仕様から想定できる動作に対しては
> 問題なさそうに見えます。

最初のはいろいろ想定漏れがありました。
いろいろ指摘ありがとうございます！

> ループの中と外で同じチェックをしているので、
...
> という感じでまとめるともっといいかなと思いました。
> 最後もスッキリと
...
> でいいかなとか。

とてもスッキリで良いですね。
そのまま置き換えたパッチを添付しておきます。
テストも全部 OK でした。

> it も乙でした！

どうもです！

-- 
Jun Futagawa
-------------- next part --------------
テキスト形式以外の添付ファイルを保管しました...
ファイル名: s2tiger-s2jdbc-includes-excludes4.patch
型:         text/x-patch
サイズ:     15595 バイト
説明:       無し
URL:        <http://ml.seasar.org/archives/seasar-user/attachments/20130714/1bec88b2/attachment.bin>