From aeb5824c52a45059cc2462cef9e52a73e2831e59 Mon Sep 17 00:00:00 2001 From: yugui Date: Fri, 5 Dec 2008 13:35:06 +0000 Subject: merges r20547 from trunk into ruby_1_9_1. * rational.c (nurat_{to_s,inspect}): performance improvement. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_9_1@20558 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- rational.c | 32 +++++++++++++++++++++++--------- 1 file changed, 23 insertions(+), 9 deletions(-) (limited to 'rational.c') diff --git a/rational.c b/rational.c index d7966a1e7a..3c29aab973 100644 --- a/rational.c +++ b/rational.c @@ -27,8 +27,8 @@ VALUE rb_cRational; static ID id_abs, id_cmp, id_convert, id_equal_p, id_expt, id_floor, - id_format, id_hash, id_idiv, id_inspect, id_integer_p, id_negate, - id_to_f, id_to_i, id_to_s, id_truncate; + id_hash, id_idiv, id_inspect, id_integer_p, id_negate, id_to_f, + id_to_i, id_to_s, id_truncate; #define f_boolcast(x) ((x) ? Qtrue : Qfalse) @@ -1111,19 +1111,34 @@ nurat_hash(VALUE self) } static VALUE -nurat_to_s(VALUE self) +nurat_format(VALUE self, VALUE (*func)(VALUE)) { + VALUE s; get_dat1(self); - return rb_funcall(rb_mKernel, id_format, 3, - rb_str_new2("%d/%d"), dat->num, dat->den); + + s = (*func)(dat->num); + rb_str_cat2(s, "/"); + rb_str_concat(s, (*func)(dat->den)); + + return s; +} + +static VALUE +nurat_to_s(VALUE self) +{ + return nurat_format(self, f_to_s); } static VALUE nurat_inspect(VALUE self) { - get_dat1(self); - return rb_funcall(rb_mKernel, id_format, 3, - rb_str_new2("(%d/%d)"), dat->num, dat->den); + VALUE s; + + s = rb_str_new2("("); + rb_str_concat(s, nurat_format(self, f_inspect)); + rb_str_cat2(s, ")"); + + return s; } static VALUE @@ -1484,7 +1499,6 @@ Init_Rational(void) id_equal_p = rb_intern("=="); id_expt = rb_intern("**"); id_floor = rb_intern("floor"); - id_format = rb_intern("format"); id_hash = rb_intern("hash"); id_idiv = rb_intern("div"); id_inspect = rb_intern("inspect"); -- cgit v1.2.3