From: mail@...
Date: 2014-11-04T05:47:07+00:00
Subject: [ruby-core:66074] [ruby-trunk - Feature #10474] [Open] [PATCH 1/1] Refactoring math.c (Combined some macros into one macro)

Issue #10474 has been reported by gogo tanaka.

----------------------------------------
Feature #10474: [PATCH 1/1] Refactoring math.c (Combined some macros into one macro)
https://bugs.ruby-lang.org/issues/10474

* Author: gogo tanaka
* Status: Open
* Priority: Normal
* Assignee: 
* Category: 
* Target version: 
----------------------------------------
Hi, there.

We call `Need_Float` macro with `RFLOAT_VALUE` macro at the all time,
so we don't need to assign result of `rb_to_float(x)` to temporary variable.

This is main reason I combined these two macros into one macro called `Get_RFloat_Value` and 
removed `Need_Float2`.

I'm not sure the name `Get_RFloat_Value` is best.
If you have more preferable one, let me know and I'll fix patch :)

Thanks. gogo.

---Files--------------------------------
combine_some_macros_into_one_at_math.c.patch (7.19 KB)


-- 
https://bugs.ruby-lang.org/