Hiroyuki OYAMA
oyama****@modul*****
2005年 4月 10日 (日) 04:37:18 JST
module.jpの小山です。 # すんません、変なエンコードでメール送ってましたので再送し # ます(汗) ドキュメントとsenna.hにはあるけど実装されていない const void *sen_records_curr_key(sen_records *records); を実装してみました。 http://module.jp/dist/senna-rev9.sen_records_curr_key.diff sen_records_next()がkeyを返すので不要な気もするのですが、 「書いてあるのに使えない」というのが気になったもので :-) ただ書きながら思ったのですが、lib/index.cとlib/set.cでnext系 と似たようなコードが重複して存在することになってしまうので、 API的に無しにするのもアリかなとも思いました。 あと、revision 7から最初に sen_init(); を実行する必要がありますよね。この点ドキュメントの更新、ま たはsen_index_open()等で自動的にcallするなどして頂けたりす ると嬉しいです。 # 最初忘れててハマりました。 __ 重要ではないのですが test/Makefile.am で /usr/lib/libmecab.la を直接指定しているのは何故ですか? configure.acで AC_CHECK_LIB(mecab, mecab_new3)しているのでlibsenna.laに dependency_libsとして自動的に追加されるので不要なのではない かと思います。 # libmecab.a しか無い場合に困るのかしら? 私の場合 /usr/local/lib/libmecab.* にインストールしているの で、アップデート時に引っかかって気持ちが悪いというだけの下ら ない理由ですので無視して頂いてかまいませんです。 # 関係ないけどMeCab付属のlibtool関連スクリプト古いなぁ... ___ Hiroyuki OYAMA <oyama****@modul*****>, <oyama****@cpan*****> Hackしてる？ ----> http://module.jp/book/
