Re: [MD:6405] PDUMP_OBJECT_ARRAY_SIZE

[MIYOSHI Masanori <miyoshi@xxxxxxxxxxx>]

>>>>> [meadow-develop : No.6407] にて
>>>>> 藤井さんは書きました:
藤井> この PDUMP_OBJECT_ARRAY_SIZE ですが、以下のような不具合をまれに発生さ
藤井> せるので、固定値を makefile で指定するのは良くないんじゃないかと思います。

藤井> 	- VC で増量したけど Cygwin では増量し忘れてエラー
藤井> 	- make all では問題ないけど、make bootstrap ではサイズ不足でエラー

固定値が良くないのは同感です。もし、固定値のままにするにしても、
alloc.c に埋め込んだ方が、まだましだと思います。


藤井> src/alloc.c を見てみたところ、PDUMP_OBJECT_ARRAY_SIZE のサイズを必要最
藤井> 小限より大きくしても、meadow.dmp のサイズや通常の実行時のメモリサイズ
藤井> には悪影響はないと判断しました。そこで、配列のサイズが足りなかったら内
藤井> 部で増量する仕組を追加するパッチを作りました。

藤井> portable dumper の仕組みを良く分かっている訳ではないので、思い違いがあ
藤井> るかもしれませんが、宜しければ使ってみてください。

良いと思うのですが、私も portable dumper の仕組みを良く分かってい
る訳ではないので、確信はありません。

コメントしてもらえませんか? > 実装した林さんか永野さん

コメントをもらえないようであれば、まずは、2.20-dev に commit して
もらって、様子を見ましょうか。


藤井> # あと、パッチを作成しているときに気になったのですが、
藤井> # pdump_lisp_object に格納されている構造体の index メンバと size メン
藤井> # バが初期化されていないように見えました。問題ないのでしょうか？

むむ。確かに初期化されてないように見えますね。ゼロクリアで良いのか
な?

--
三好 雅則 http://www.boreas.dti.ne.jp/~miyoshi/