Forum: Ruby-core [Assigned] Hide Rational internal

31e9336577dcba34f9ea8025a763c900?d=identicon&s=25 unknown (Guest)
on 2014-05-02 13:33
(Received via mailing list)
Issue #9513 has been updated by tadayoshi funaba.

Status changed from Open to Assigned
Assignee changed from tadayoshi funaba to Yukihiro Matsumoto

i'm ok.


----------------------------------------
Feature #9513: Hide Rational internal
https://bugs.ruby-lang.org/issues/9513#change-46457

* Author: Akira Tanaka
* Status: Assigned
* Priority: Normal
* Assignee: Yukihiro Matsumoto
* Category:
* Target version:
----------------------------------------
I'd like to hide C-level Ratinal internal.
I made a patch to do it.
This is similar to [Feature #6083] for struct RBignum.

If the internal is hidden, we can change it without compatibility
problem.
For example, we can use GMP.

The patch moves struct RRational and related macros to internal.h.
It also provides two accessors, rb_rational_num and rb_rational_den and
changes ext/bigdecimal/bigdecimal.c and ext/date/date_core.c to
use these accessors.

How do you think, tadf?


---Files--------------------------------
hide-rational-internal.patch (5.98 KB)
This topic is locked and can not be replied to.