morita
morit****@razil*****
2008年 9月 16日 (火) 08:08:48 JST
森です。
本件修正しました。
あと、sen_obj_close()でsen_spaceを解放できるようにしました。
添付のパッチを当てるとunit testでのメモリリークが少なくなります。
morita さんは書きました:
> 森です。
>
> これはsennaのバグっぽいですね。。追ってみます。
> ご指摘ありがとうございます。。
>
> Kouhei Sutou さんは書きました:
>> 須藤です。
>>
>> けっこう自信がないのですが、sen_ii_update()の動作がおかしそうなところを
>> 見つけました。原因までは追っていません。ごめんなさい。
>>
>> test/unit/test-inverted-index.c:test_crud_may_be_bug()を見てほしいのですが、
>> テストに関係のないはずのレコードID 4のための行を削除したりすると動きが
>> 変わります。手元だと、そのまま動かすとテストが失敗します。
>>
>> 1) Failure: test_crud_may_be_bug
>> <NULL == retrieve_record_ids("検索")>
>> expected: <()>
>> but was: <("3")>
>> ./test-inverted-index.c:464: test_crud_may_be_bug()
>>
>> しかし、
>>
>> add_data(4, 1, "INSTALL.JA"); /* this test will be passed if this line
>> is removed */
>>
>> という行を削除するとテストがパスするようになります。
>>
>> またテストの方が悪いのかどうかはわからないのですがとりあえず
>> 報告しておきます。
>>
>> ところで、test/unit/stress/test-stress-inverted-index.cも必要ですか？
>>
>> _______________________________________________
>> Senna-dev mailing list
>> Senna****@lists*****
>> http://lists.sourceforge.jp/mailman/listinfo/senna-dev
>> バグ報告方法：http://qwik.jp/senna/bug_report.html
>>
>
> _______________________________________________
> Senna-dev mailing list
> Senna****@lists*****
> http://lists.sourceforge.jp/mailman/listinfo/senna-dev
> バグ報告方法：http://qwik.jp/senna/bug_report.html
>
-------------- next part --------------
テキスト形式以外の添付ファイルを保管しました...
ファイル名: tmp.patch
型: text/x-diff
サイズ: 2264 バイト
説明: 無し
Download