[Senna-dev 1042] Re: 動的にメモリ割り当て関数を変更

Back to archive index

Tasuku SUENAGA a****@razil*****
2008年 9月 28日 (日) 15:25:06 JST


末永です。

>> 現在のconfigureオプションである--enable-fmallocをつけたときだけ
>> 有効になるようになると嬉しいなぁ…と。
> そういうようにしてコミットしてみました。

ありがとうございます!試します。

失敗するテストについては
もうちょっと再現条件を絞ってみますね。

# そういえば、こちらもコメントしておきました。
  http://d.hatena.ne.jp/nicodic/20080923/word_blog_search#c1222581803

Kouhei Sutou さんは書きました:
> 須藤です。
> 
> In <48DCB****@razil*****>
>   "[Senna-dev 1038] Re: 動的にメモリ割り当て関数を変更" on Fri, 26 Sep 2008 18:58:25 +0900,
>   Tasuku SUENAGA <a****@razil*****> wrote:
> 
>> というわけで、以下のパッチについて
>> 現在のconfigureオプションである--enable-fmallocをつけたときだけ
>> 有効になるようになると嬉しいなぁ…と。
> 
> そういうようにしてコミットしてみました。
> 試してみてもらってもよいですか?
> 
>> ---
>> CRASH!!!
>> #5  0xb7762ef5 in trie_test_data_newv (
>> #6  0xb7764895 in test_data_new (key=0xb77734f1 "?\202\211?\203\213?\202?",
>> #7  0xb776a2a5 in data_delete () at test-patricia-trie.c:340
>> #8  0xb7f9bb81 in run (test=0x808a808, test_context=0x808b1f0,
>> #9  0xb7f98e5f in cut_test_run (test=0x808a808, test_context=0x808b1f0,
>> #10 0xb7f9c74e in cut_test_case_run_with_filter (test_case=0x8062670,
>> #11 0xb7f9cfd0 in run (data=0x80b0538) at cut-test-suite.c:242
>> #12 0xb7f9d6dc in cut_test_suite_run_test_cases (test_suite=0x805e808,
>> #13 0xb7f9da09 in cut_test_suite_run_with_filter (test_suite=0x805e808,
>> #14 0xb7f9da96 in cut_test_suite_run (suite=0x805e808, run_context=0x8059800)
>> #15 0xb7f96f58 in runner_run (runner=0x8059800) at cut-test-runner.c
>> (gdbだと)
>> Program received signal SIGSEGV, Segmentation fault.
>> [Switching to Thread 0xb7cbf700 (LWP 24536)]
>> 0xb775eef5 in trie_test_data_newv (key=0xb776f4f1 "?\202\211?\203\213?\202?",
>>     search_key=0x0, expected_key=0x0, expected_rc=sen_success,
>>     expected_strings=0x0, increment=0,
>>     set_parameters=0xb775f0f5 <set_sis_and_utf8_encoding>, args=0x0)
>>     at test-patricia-trie.h:300
>> 300         if (!*args)
> 
> あぁ、これはテストの書き方が悪かったです。。。
> 修正しておきました。
> 
>> 以下1) 2)以外は32bit環境のみで起こる。
>>
>> 1) Omission: test_create_with_null_path
>> sen_ii_create() doesn't check NULL path. should be checked?
>> ./test-inverted-index.c:206: test_create_with_null_path()
> 
> これは省略したテストを表示しているだけなので問題ないですね。
> 
>> 2) Failure: test_crud
>> <gcut_take_new_list_string("1", "2", "3", NULL) == retrieve_record_ids("検索")>
>> expected: <("1", "2", "3")>
>>  but was: <()>
>> ./test-inverted-index.c:399: test_crud()
> 
> このあたりははこの間、森さんに直してもらったところだった気が
> します。手元だとパスするんですよねぇ。。。
---
tasuku <a****@razil*****>




Senna-dev メーリングリストの案内
Back to archive index