[ruby-dev:36946] Re: {Complex,Rational}::Unify

こんにちは、なかむら(う)です。

In message "[ruby-dev:36926] Re: {Complex,Rational}::Unify"
    on Oct.25,2008 15:00:51, <[e-mail]> wrote:
> シンボルの衝突も含めて、プリプロセッサマクロでmathn版と組み込み版を切り
> 替えてコンパイルするようにパッチを書いてみました。

yuguiさんのパッチそのままでは解決してないようだったので、その
他微細な修正も加えてyuguiさんのパッチへの差分を作りました。
具体的には

  * rb_cComplex、rb_cRationalはruby.hでexternされてるので.c側
    で宣言自体が不要(というか邪魔)なので本体側の定義のみ残し
    てext側の宣言を削除。

  * ext側の.cは本体側の.cに依存してるのでdependを追加。

  * complex.c (rb_Complex) の戻り値返し忘れを修正。

です。

--- complex.c.orig	2008-10-27 09:29:07.440873000 +0900
+++ complex.c	2008-10-27 09:33:48.840873200 +0900
@@ -20,8 +20,7 @@
 #define TWO INT2FIX(2)
 
-#ifdef EXT_MATHN
-extern
-#endif
+#ifndef EXT_MATHN
 VALUE rb_cComplex;
+#endif
 
 static ID id_abs, id_abs2, id_arg, id_cmp, id_conj, id_convert,
@@ -983,5 +982,5 @@ VALUE
 rb_Complex(VALUE x, VALUE y)
 {
-    rb_funcall(rb_cComplex, id_convert, 2, x, y);
+    return rb_funcall(rb_cComplex, id_convert, 2, x, y);
 }
 #endif
--- rational.c.orig	2008-10-27 09:36:57.654873000 +0900
+++ rational.c	2008-10-27 09:37:11.544873200 +0900
@@ -25,8 +25,7 @@
 #define TWO INT2FIX(2)
 
-#ifdef EXT_MATHN
-extern
-#endif
+#ifndef EXT_MATHN
 VALUE rb_cRational;
+#endif
 
 static ID id_abs, id_cmp, id_convert, id_equal_p, id_expt, id_floor,
--- /dev/null	2006-12-01 09:00:00.000000000 +0900
+++ ext/mathn/complex/depend	2008-10-27 09:36:04.195873200 +0900
@@ -0,0 +1 @@
+complex.o: ../../../complex.c
--- /dev/null	2006-12-01 09:00:00.000000000 +0900
+++ ext/mathn/rational/depend	2008-10-27 09:36:32.940873200 +0900
@@ -0,0 +1 @@
+rational.o: ../../../rational.c

それでは。
-- 
U.Nakamura <[e-mail]>