Kouhei Sutou
kou****@clear*****
Mon Oct 7 10:28:49 JST 2013
In <52520C1D.2030802 �� clear-code.com> "Re: [Groonga-commit] groonga/groonga �� 5df7b51 [master] mruby: put an argument in a line" on Mon, 07 Oct 2013 10:19:25 +0900, YUKI Hiroshi <yuki �� clear-code.com> wrote: > > > mruby: put an argument in a line > > > > Because other arugments are in each line. > > > > とのことですが、 > > > mrb_sis = mrb_funcall(mrb, mrb_obj_value(ctx->impl->mrb.module), > "build", 4, > > この部分も各行に分けた方がよいという事にならないでしょうか？ うーん、個人的には mrb_sis = mrb_funcall(mrb, mrb_obj_value(ctx->impl->mrb.module), "build", 4, までならわけてもいいかなぁと思います。 というのは、 mrb_obj_value(ctx->impl->mrb.module), "build", 4, の引数はひとまとまりなんですよ。 「mrb_obj_value(ctx->impl->mrb.module)」オブジェクトの 「"build"」メソッドを 「4」つの引数で呼びます。 みたいな意味です。↑でメソッド呼び出しの情報を表現しているの でできるだけまとめてあげたいなぁというのがあります。 で、それ以降は具体的な引数で、ひとつにまとめられるほど短くも ないので mrb_cptr_value(mrb, expr), mrb_cptr_value(mrb, n), mrb_fixnum_value(op), mrb_fixnum_value(size)); よりは mrb_cptr_value(mrb, expr), mrb_cptr_value(mrb, n), mrb_fixnum_value(op), mrb_fixnum_value(size)); が統一感があっていいなぁと思いました。 ということで、引数の中でも同じ分類のものはひとかたまりにして あげたいのでそこはわけたくないなぁという感じです。 似たようなのだと文字列のアドレスと長さを渡す string_ptr, string_len みたいなやつも、できればひとまとまりにしたい（同じ行に入れて あげたい）です。 > Kouhei Sutou wrote: >> Author >> Kouhei Sutou <kou �� clear-code.com> >> Date >> 2013-10-06 10:58:11 +0900 (Sun, 06 Oct 2013) >> New Revision >> 5df7b514671c912fb0007c2b1740e08e8f295aea >> <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea> >> Message >> >> mruby: put an argument in a line >> >> Because other arugments are in each line. >> >> Modified files >> >> * lib/mrb/mrb_expr.c >> <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#diff-0> >> >> Modified: lib/mrb/mrb_expr.c (+2 -1) >> =================================================================== >> >> ... <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0L358> >> 359 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0L359> >> 360 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0L360> >> 361 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0L361> >> 362 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0L362> >> >> >> 363 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0L363> >> 364 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0L364> >> 365 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0L365> >> >> >> >> ... <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0R358> >> 359 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0R359> >> 360 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0R360> >> 361 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0R361> >> >> 362 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0R362> >> 363 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0R363> >> 364 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0R364> >> 365 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0R365> >> 366 <https://github.com/groonga/groonga/commit/5df7b514671c912fb0007c2b1740e08e8f295aea#L0R366> >> >> >> >> @@ -359,7 +359,8 @@grn_mrb_scan_info_build(grn_ctx *ctx, grn_obj *expr, int *n, >> >> mrb_sis = mrb_funcall(mrb, mrb_obj_value(ctx->impl->mrb.module), "build", 4, >> mrb_cptr_value(mrb, expr), >> - mrb_cptr_value(mrb, n), mrb_fixnum_value(op), >> + mrb_cptr_value(mrb, n), >> + mrb_fixnum_value(op), >> mrb_fixnum_value(size)); >> sis = mrb_cptr(mrb_sis); >> return sis; >> >> >> >> _______________________________________________ >> Groonga-commit mailing list >> Groonga-commit �� lists.sourceforge.jp >> http://lists.sourceforge.jp/mailman/listinfo/groonga-commit >> > > -- > 結城 洋志 <YUKI Hiroshi> > E-mail: yuki �� clear-code.com > > 株式会社クリアコード > 〒113-0033 東京都文京区本郷3-27-12 > 本郷デントビル2階 > TEL : 03-6231-7270 > FAX : 03-6231-7271 > WWW : http://www.clear-code.com/ > > _______________________________________________ > Groonga-commit mailing list > Groonga-commit �� lists.sourceforge.jp > http://lists.sourceforge.jp/mailman/listinfo/groonga-commit
