Mosh/レビュー


Top / Mosh / レビュー

レビュー

r1146

FileBinaryOutputPort についてソースを読んでいます。気になった点がいくつかあるので書いておきます。重要な処理なのでいつもより細かめに見ています。

FileBinaryInputPort

r740

OK。todo で目印つけるのはとても良い。

rev715 他

rev710

除算、剰余算 OK です。

rev707

OKです。

rev695

1点だけ。

const int ret = fx1 + fx2;

int よりも int32_t の方が意図が明確に伝わると思うのでそこだけ修正をお願いします。

rev690

軽くしましょう

fx+ は数値が Fixnum の範囲である事が分かっているので計算が速くなるのを期待できます。
なので Bignum::add を使うと

以下のような方法がよいかも。
Fixnum の数値は 29bit なので足し算の結果は 30bit に収まる事を利用して

const long ret = n1.toFixnum() + n2.toFixnum();
if (Fixnum::canFit(ret)) {
   return Object::makeFixnum(ret);
} else {
           エラー
}

エラーについて

callImplementationRestrictionViolationImmidiaImmediately
まず名前に Immediately が若干重複しています。

あと Immediately 系ではなく例に出したように void scheme::callLexicalViolationAfter 。
After 系を使って欲しいです。
説明していなかったので申し訳ないのですが

あとは

callImplementationRestrictionViolationImmidiaImmediately("fx+", "sum is not a fixnum");

r685

r677

rev668

rev662

rev639/640

rev636

コメント

コメントはありません。 コメント/Mosh/レビュー?

お名前:

MENU

now: 6

リンク


最新の20件
2020-06-30 2020-06-26 2020-06-15 2020-05-27 2020-05-22 2020-05-14 2020-05-12 2020-05-10
最新の20件
2010-02-01 2010-01-31 2010-01-30 2010-01-29 2010-01-16

Counter: 2239, today: 1, yesterday: 1

添付ファイル: fileピクチャ 8.png 519件 [詳細]

リロード   新規 編集 凍結 差分 添付 複製 改名   トップ 一覧 検索 最終更新 バックアップ   ヘルプ   最終更新のRSS

Last-modified: 2009-02-05 (木) 23:31:45 (4248d);  Modified by mona
PukiWiki 1.4.6 Copyright © 2001-2005 PukiWiki Developers Team. License is GPL.
Based on "PukiWiki" 1.3 by yu-ji
Powered by PHP 5.2.17
HTML convert time to 0.100 sec.