From: ayumu.aizawa@...
Date: 2015-01-03T15:13:29+00:00
Subject: [ruby-core:67312] [ruby-trunk - Bug #10487] [PATCH 3/3] More than 3 arguments Passing to CMath.log doesn't make sense.

Issue #10487 has been updated by Ayumu AIZAWA.


Hi Tanaka-san
Please post new patch which was applied Marc's review. 
I would help you for assign the right person of ruby-core for this issue.
Regards.


----------------------------------------
Bug #10487: [PATCH 3/3] More than 3 arguments Passing to CMath.log doesn't make sense.
https://bugs.ruby-lang.org/issues/10487#change-50772

* Author: gogo tanaka
* Status: Open
* Priority: Normal
* Assignee: 
* Category: 
* Target version: 
* ruby -v: 2.2.0dev
* Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN
----------------------------------------
Hi, there. 

I was wondering which is prefer reporting this issue as bugs or feature.
Finally I decided to report as bugs because I thought such a issue being totally unexpected can be called bugs.

Aside from that, let me show the before my patch and after my patch.

## Before
```
CMath.log(1i,1,1,1,1,1,1,1,1)
# => (NaN+Infinity*i)
```

## After
```
CMath.log(1i,1,1,1,1,1,1,1,1)
# => ArgumentError: wrong number of arguments (9 for 1..2)
```

`def log(z, b=E)` may look little bit strange, but I assert for all num `log(num)` equal `log(num, E)`

and checking second argument passing or not takes pain.

Thanks. gogo.

---Files--------------------------------
modify_cmath.rb.patch (1.15 KB)
adjust_test_complex.rb.patch (955 Bytes)
update_NEWS.patch (629 Bytes)


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