From b60a7b43feaed02089c158faf0bf647327ed61f0 Mon Sep 17 00:00:00 2001 From: mrkn Date: Fri, 18 Mar 2016 14:52:46 +0000 Subject: * numeric.c (int_to_f, fix_to_f): rename fix_to_f to int_to_f, and add treatment for subclasses which don't have definitions of to_f method. * numeric.c (Integer#to_f, Fixnum#to_f): move to_f method from Fixnum to Integer. * ext/-test-/integer/my_integer.rb: define helper class for testing to_f method for a subclass of Integer. * ext/-test-/integer/extconf.rb: ditto. * ext/-test-/integer/init.c: ditto. * test/-ext-/integer/test_my_integer.rb: examine to_f method for a subclass of Integer. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@54179 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 18 ++++++++++++++++++ ext/-test-/integer/extconf.rb | 8 ++++++++ ext/-test-/integer/init.c | 11 +++++++++++ ext/-test-/integer/my_integer.c | 16 ++++++++++++++++ numeric.c | 15 ++++++++++----- test/-ext-/integer/test_my_integer.rb | 26 ++++++++++++++++++++++++++ 6 files changed, 89 insertions(+), 5 deletions(-) create mode 100644 ext/-test-/integer/extconf.rb create mode 100644 ext/-test-/integer/init.c create mode 100644 ext/-test-/integer/my_integer.c create mode 100644 test/-ext-/integer/test_my_integer.rb diff --git a/ChangeLog b/ChangeLog index 442025d0d4..9830f370bb 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,21 @@ +Fri Mar 18 23:41:00 2016 Kenta Murata + + * numeric.c (int_to_f, fix_to_f): rename fix_to_f to int_to_f, and add + treatment for subclasses which don't have definitions of to_f method. + + * numeric.c (Integer#to_f, Fixnum#to_f): move to_f method from Fixnum + to Integer. + + * ext/-test-/integer/my_integer.rb: define helper class for testing + to_f method for a subclass of Integer. + + * ext/-test-/integer/extconf.rb: ditto. + + * ext/-test-/integer/init.c: ditto. + + * test/-ext-/integer/test_my_integer.rb: examine to_f method for a + subclass of Integer. + Fri Mar 18 22:32:00 2016 Kenta Murata * include/ruby/intern.h (rb_big_hash): Move to internal.h. diff --git a/ext/-test-/integer/extconf.rb b/ext/-test-/integer/extconf.rb new file mode 100644 index 0000000000..20bfffbec2 --- /dev/null +++ b/ext/-test-/integer/extconf.rb @@ -0,0 +1,8 @@ +# frozen_string_literal: false +$INCFLAGS << " -I$(topdir) -I$(top_srcdir)" +$srcs = Dir[File.join($srcdir, "*.{#{SRC_EXT.join(%q{,})}}")] +inits = $srcs.map {|s| File.basename(s, ".*")} +inits.delete("init") +inits.map! {|s|"X(#{s})"} +$defs << "-DTEST_INIT_FUNCS(X)=\"#{inits.join(' ')}\"" +create_makefile("-test-/integer") diff --git a/ext/-test-/integer/init.c b/ext/-test-/integer/init.c new file mode 100644 index 0000000000..fc256ea16b --- /dev/null +++ b/ext/-test-/integer/init.c @@ -0,0 +1,11 @@ +#include "ruby.h" + +#define init(n) {void Init_##n(VALUE klass); Init_##n(klass);} + +void +Init_integer(void) +{ + VALUE mBug = rb_define_module("Bug"); + VALUE klass = rb_define_class_under(mBug, "Integer", rb_cObject); + TEST_INIT_FUNCS(init); +} diff --git a/ext/-test-/integer/my_integer.c b/ext/-test-/integer/my_integer.c new file mode 100644 index 0000000000..d86474bd7d --- /dev/null +++ b/ext/-test-/integer/my_integer.c @@ -0,0 +1,16 @@ +#include "ruby.h" + +static VALUE +my_integer_s_new(VALUE klass) +{ + return Data_Wrap_Struct(klass, 0, 0, 0); +} + +void +Init_my_integer(VALUE klass) +{ + VALUE cMyInteger; + + cMyInteger = rb_define_class_under(klass, "MyInteger", rb_cInteger); + rb_define_singleton_method(cMyInteger, "new", my_integer_s_new, 0); +} diff --git a/numeric.c b/numeric.c index fc2a514201..e0871049f7 100644 --- a/numeric.c +++ b/numeric.c @@ -3765,18 +3765,23 @@ fix_aref(VALUE fix, VALUE idx) /* * call-seq: - * fix.to_f -> float + * int.to_f -> float * - * Converts +fix+ to a Float. + * Converts +int+ to a Float. * */ static VALUE -fix_to_f(VALUE num) +int_to_f(VALUE num) { double val; - val = (double)FIX2LONG(num); + if (FIXNUM_P(num)) { + val = (double)FIX2LONG(num); + } + else { + rb_raise(rb_eTypeError, "Unknown subclass for to_f: %s", rb_obj_classname(num)); + } return DBL2NUM(val); } @@ -4214,6 +4219,7 @@ Init_Numeric(void) rb_define_method(rb_cInteger, "ord", int_ord, 0); rb_define_method(rb_cInteger, "to_i", int_to_i, 0); rb_define_method(rb_cInteger, "to_int", int_to_i, 0); + rb_define_method(rb_cInteger, "to_f", int_to_f, 0); rb_define_method(rb_cInteger, "floor", int_to_i, 0); rb_define_method(rb_cInteger, "ceil", int_to_i, 0); rb_define_method(rb_cInteger, "truncate", int_to_i, 0); @@ -4253,7 +4259,6 @@ Init_Numeric(void) rb_define_method(rb_cFixnum, "<<", rb_fix_lshift, 1); rb_define_method(rb_cFixnum, ">>", rb_fix_rshift, 1); - rb_define_method(rb_cFixnum, "to_f", fix_to_f, 0); rb_define_method(rb_cFixnum, "size", fix_size, 0); rb_define_method(rb_cFixnum, "bit_length", rb_fix_bit_length, 0); rb_define_method(rb_cFixnum, "succ", fix_succ, 0); diff --git a/test/-ext-/integer/test_my_integer.rb b/test/-ext-/integer/test_my_integer.rb new file mode 100644 index 0000000000..bc4757f463 --- /dev/null +++ b/test/-ext-/integer/test_my_integer.rb @@ -0,0 +1,26 @@ +# frozen_string_literal: false +require 'test/unit' +require "-test-/integer" + +class TestIntegerExt < Test::Unit::TestCase + def test_my_integer_to_f + assert_raise(TypeError) do + Bug::Integer::MyInteger.new.to_f + end + + begin + Bug::Integer::MyInteger.class_eval do + def to_f + end + end + + assert_nothing_raised do + Bug::Integer::MyInteger.new.to_f + end + ensure + Bug::Integer::MyInteger.class_eval do + remove_method :to_f + end + end + end +end -- cgit v1.2.3