Ryo Onodera
onode****@clear*****
2012年 4月 18日 (水) 18:11:50 JST
小野寺です。 > コメントありがとうございます。 お役に立ててなによりです！ > 別途指摘していただいたようにthe value ofに置き換えるのが良さそうですね。 了解しました！ On Wed, 2012-04-18 at 17:48 +0900, HAYASHI Kentaro wrote: > To: 小野寺さん > > 林です。 > > コメントありがとうございます。 > > 修正案についてなのですが、言いたかったことは > 「そのテストは、output_columnsオプションの引数に含まれる空白が > 無視されるかを確認する。」というものなので、 > 別途指摘していただいたようにthe value ofに置き換えるのが良さそうですね。 > > The test checks the value of output_columns option ignores spaces. > > 以上です。 > > On Wed, 18 Apr 2012 15:47:41 +0900 > Ryo Onodera <onode****@clear*****> wrote: > > > 小野寺です。 > > > > > The test checks output_columns option ignores spaces in argument. > > > > 言いたいことは伝わるのですが、文章的に少しおかしいところが3つありまし > > た。 > > > > 1. in arguemntについて > > > > この文を日本語に戻すと、「そのテストは、output_columnsオプションが空白を > > 引数の中で無視するか、を確認する」となります。「空白を引数の中で無視す > > る」というのはおかしい気がしています。 > > > > おそらく、「引数のoutput_columnオプションの中にある空白を無視する」と言 > > いたかったのだと思います。 > > > > 説明すると一般的な英文の解釈では、「in argument」は動詞である「ignores」 > > を修飾します。「output_columns option」を修飾するためには別の表現にする > > 必要があります。その場合は以下のようにした方がいいです。 > > > > The test checks output_columns option of argument ignores spaces. > > (そのテストは、引数のoutput_columnsオプションが空白を無視するか、を確認 > > する。) > > > > 2. argumentについて > > > > これはselectコマンドの引数という意味でのargumentだと思いますが、その場合 > > はselectコマンドの色々な引数を集合的に指し示しているので、複数形にすべき > > だと思います。 > > > > 3. output_columns optionについて > > > > これは名詞であり、冠詞をつけるべきではない例外でもありませんので、冠詞を > > つける必要があります。この場合は文脈上、特定のものを限定して指し示すため > > に登場しているので、定冠詞をつけるべきだと思います。 > > > > > > > > 1.と2.と3.を合わせ、最終的には次のようになります。 > > > > The test checks the output_columns option of arguments ignores spaces. > > (そのテストは、引数のoutput_columnsオプションが空白を無視するか、を確認 > > する。) > > > > On Wed, 2012-04-18 at 14:26 +0900, null+****@clear***** wrote: > > > khayashi 2012-04-18 14:26:34 +0900 (Wed, 18 Apr 2012) > > > > > > New Revision: 578530d4f51f986e8a6dc576c062e3b93e13bf7d > > > > > > Log: > > > test: migrate a test from Cutter to grntest style > > > > > > The test checks output_columns option ignores spaces in argument. > > > > > > Added files: > > > test/function/suite/select/output-columns/with-space.expected > > > test/function/suite/select/output-columns/with-space.test > > > Modified files: > > > test/unit/core/test-command-select.c > > > > > > Added: test/function/suite/select/output-columns/with-space.expected (+44 -0) 100644 > > > =================================================================== > > > --- /dev/null > > > +++ test/function/suite/select/output-columns/with-space.expected 2012-04-18 14:26:34 +0900 (6cb56dc) > > > @@ -0,0 +1,44 @@ > > > +table_create Sites TABLE_HASH_KEY ShortText > > > +[[0,0.0,0.0],true] > > > +column_create Sites uri COLUMN_SCALAR ShortText > > > +[[0,0.0,0.0],true] > > > +load --table Sites > > > +[ > > > +["_key","uri"], > > > +["groonga","http://groonga.org/"], > > > +["razil","http://razil.jp/"] > > > +] > > > +[[0,0.0,0.0],2] > > > +select Sites --output_columns '_key, uri' > > > +[ > > > + [ > > > + 0, > > > + 0.0, > > > + 0.0 > > > + ], > > > + [ > > > + [ > > > + [ > > > + 2 > > > + ], > > > + [ > > > + [ > > > + "_key", > > > + "ShortText" > > > + ], > > > + [ > > > + "uri", > > > + "ShortText" > > > + ] > > > + ], > > > + [ > > > + "groonga", > > > + "http://groonga.org/" > > > + ], > > > + [ > > > + "razil", > > > + "http://razil.jp/" > > > + ] > > > + ] > > > + ] > > > +] > > > > > > Added: test/function/suite/select/output-columns/with-space.test (+11 -0) 100644 > > > =================================================================== > > > --- /dev/null > > > +++ test/function/suite/select/output-columns/with-space.test 2012-04-18 14:26:34 +0900 (ec538d4) > > > @@ -0,0 +1,11 @@ > > > +table_create Sites TABLE_HASH_KEY ShortText > > > +column_create Sites uri COLUMN_SCALAR ShortText > > > + > > > +load --table Sites > > > +[ > > > +["_key","uri"], > > > +["groonga","http://groonga.org/"], > > > +["razil","http://razil.jp/"] > > > +] > > > + > > > +select Sites --output_columns '_key, uri' > > > > > > Modified: test/unit/core/test-command-select.c (+0 -21) > > > =================================================================== > > > --- test/unit/core/test-command-select.c 2012-04-17 11:48:37 +0900 (5a04907) > > > +++ test/unit/core/test-command-select.c 2012-04-18 14:26:34 +0900 (def9c88) > > > @@ -24,7 +24,6 @@ > > > > > > #include "../lib/grn-assertions.h" > > > > > > -void test_output_columns_with_space(void); > > > void test_vector_geo_point(void); > > > void test_vector_geo_point_with_query(void); > > > void test_unmatched_output_columns(void); > > > @@ -102,26 +101,6 @@ cut_teardown(void) > > > } > > > > > > void > > > -test_output_columns_with_space(void) > > > -{ > > > - assert_send_commands("table_create Sites TABLE_HASH_KEY ShortText\n" > > > - "column_create Sites uri COLUMN_SCALAR ShortText\n" > > > - "load --table Sites\n" > > > - "[\n" > > > - "[\"_key\",\"uri\"],\n" > > > - "[\"groonga\",\"http://groonga.org/\"],\n" > > > - "[\"razil\",\"http://razil.jp/\"]\n" > > > - "]"); > > > - cut_assert_equal_string("[[[2]," > > > - "[[\"_key\",\"ShortText\"]," > > > - "[\"uri\",\"ShortText\"]]," > > > - "[\"groonga\",\"http://groonga.org/\"]," > > > - "[\"razil\",\"http://razil.jp/\"]]]", > > > - send_command("select Sites " > > > - "--output_columns '_key, uri'")); > > > -} > > > - > > > -void > > > test_vector_geo_point(void) > > > { > > > assert_send_command("table_create Shops TABLE_HASH_KEY ShortText"); > > > > > > _______________________________________________ > > > Groonga-commit mailing list > > > Groon****@lists***** > > > http://lists.sourceforge.jp/mailman/listinfo/groonga-commit > > > > _______________________________________________ > > Groonga-commit mailing list > > Groon****@lists***** > > http://lists.sourceforge.jp/mailman/listinfo/groonga-commit > >
