From: janosch84@...
Date: 2019-06-16T20:11:33+00:00
Subject: [ruby-core:93189] [Ruby trunk Bug#15929] Array#minmax is much slower than calling both #min and #max

Issue #15929 has been reported by janosch-x (Janosch M�ller).

----------------------------------------
Bug #15929: Array#minmax is much slower than calling both #min and #max
https://bugs.ruby-lang.org/issues/15929

* Author: janosch-x (Janosch M�ller)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
* ruby -v: 2.7.0dev
* Backport: 2.4: UNKNOWN, 2.5: UNKNOWN, 2.6: UNKNOWN
----------------------------------------
this is similar to [issue 15807 about Ranges](https://bugs.ruby-lang.org/issues/15807) and maybe also to [13917](https://bugs.ruby-lang.org/issues/13917)

current situation:

- calling `Array#minmax` incurs a performance penalty of almost 50% compared to calling both `#min` and `#max`

```ruby
require 'benchmark/ips'
arr = (1..1000).map { rand }
Benchmark.ips do |x|
  x.report('min, max') { [arr.min, arr.max] }
  x.report('minmax')   { arr.minmax }
end
```

```
min, max  53.832k (� 1.8%) i/s -  270.861k in 5.033263s
  minmax  30.093k (� 1.2%) i/s -  151.980k in 5.051078s
```

background:
- `#minmax` is included via `Enumerable`
- `Enumerable#minmax` does not call array's optimized `#min` and `#max` implementations

possible solutions:
- a) change `Enumerable#minmax` and let it `rb_funcall` `min` and `max` as suggested [here](https://bugs.ruby-lang.org/issues/15807#note-7) (will also fix 15807)
- b) implement minmax in array.c to call `rb_ary_min` and `rb_ary_max`



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

Unsubscribe: <mailto:ruby-core-request@ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>