(toppers-users 3742) Re: SSPカーネルのgenoffsetに軽微なバグ?

[杉本明加]

むらなかさん、斉藤さん

杉本です。

Perlスクリプト周りですが、ASP1.6.0辺りから流用しています。
その後ASPに対して行われた修正は反映しておりませんので
一度見直しが必要かと思っています。
# ちゃんと派生開発できていませんね…お恥ずかしい限りです

genoffsetは必要がないので次期リリースでは削除します。

以上、よろしくお願いします。

2012年1月17日14:57 Naoki Saito <nsaito.nmiri ＠ gmail.com>:
> むらなかさん
>
> 斉藤です．
>
> 1. の件ですが，genoffeset スクリプトは使われていないように思います．
> ディスパッチャが全部Cで記述されていてオフセットファイルが必要ないためです．
> 使わないファイルは基本的に置く必要が無いとは思いますが，どうしたものかと．
>
> ファイルの有無に限らず，sil.h などのような
> いろいろなカーネルに付属している単一のファイルを見ても
> SSPから見れば使わないであろうものが多くありますが削って良いものかと．．
> かといってカーネルごとにこういうファイルを作るのもムダが多いようにおもいますし．．．
>
> 2. の件については認識済みで，今後のTodo に入れております．
>
>> // use strict と use warnings を使えば簡単に検出できます。
>
> そうですね．これを使うと問題が検知しやすくなりますね．
> 検討事項に入れておきます．
>
> 以上，ご指摘ありがとうございました．
>
>
> 2012年1月17日11:18 Masaki Muranaka <monamour ＠ monaka.org>:
>> TOPPERS-Users MLのみなさま:
>> こんにちは。
>>
>> SSPカーネルを斜め読みしていて、重箱の隅的に気になるところ(バグ?)がありました。
>> 他の新世代カーネルから引き継いだ可能性もありそうな気もしますが、未確認です。
>>
>> 1. genoffset のサブルーチン parse_bit で、
>> print STDERR "genoffset: endian mismatch: $line\n";
>> なるエラーメッセージがありますが、$line は続くループ中で使われていて、
>> サブルーチン内では未定義です。
>> 関数外で定義したものを参照していると見なすには、不自然と思います。
>>
>> 2. makerelease の read_file で、
>>      open($fh, $filename) || die "Cannot open $infile";
>> とありますが、同スクリプト中では変数 infile は未定義です。
>>
>>
>> // ...といったコードの検出は、以前も本MLで指摘したかもしれませんが、
>> // use strict と use warnings を使えば簡単に検出できます。
>>
>> --
>> from もなか