Activity
From 07/03/2020 to 07/09/2020
07/09/2020
-
11:53 PM Bug #16998: [Backport] support for Bison 3 to 2.6 and 2.7
- backported into ruby_2_7 at 1ee1684a4a3edb26b5aa32acc967b9ef26284d42, 06ab2791c38c431e08bc8d9381c9a1ca727297c4, b971bad49b95d4c1f68bb2242d4267355f1cd497 and a0bb4eb04512b3f8ca50ec5fa72e3d4a8278ca46.
-
- I cannot find relevant commit with hash `8e694129305a3978313cf4bcd463c6aa2ba9ceb4`.
Maybe you meant `5c2508060b79b4299a18d05511d3530ad14b1b7a` and/or `2480ef904c66812cf366e5d9bc8a9c3c20bbf271` ? -
-
11:50 PM Revision b971bad4 (git): Bison 3.59.1 folds yydestruct function header
-
-
-
10:02 PM Bug #16982: Define instance_methods and so on explicitly on classes generated by DelegateClass
- I agree that this is a bug, and your pull request is a good way to fix it. I have squash merged your pull request. Thank you @pocke!
-
10:01 PM Bug #16982 (Closed): Define instance_methods and so on explicitly on classes generated by DelegateClass
- Applied in changeset commit:git|ba81bc24e62d03cc0f4c05cf4e2b378696631568.
----------
Add instance_methods to class generated by DelegateClass
Also, make DelegateClass.instance_method fallback to superclass.
Fixes [Bug #16982] -
- Also, make DelegateClass.instance_method fallback to superclass.
Fixes [Bug #16982] - 09:50 PM Revision 7a479b30 (git): * 2020-07-10 [ci skip]
-
09:50 PM Bug #16988 (Closed): Kernel.load loads file from current directory without '.' in path
- Applied in changeset commit:git|e1bbb9ea9ce205320267783960cfb2e9215d5f52.
----------
Document that Kernel#load will load relative to current directory [ci skip]
Update and format the Kernel#load documentation to separate the
three case... -
- TheSmartnik (Nikita Misharin) wrote in #note-2:
> nobu (Nobuyoshi Nakada) wrote in #note-1:
> ...
I don't think so. `load` has always been able to load files that exist relative to the current directory, and `require` has not. Techni... -
- Update and format the Kernel#load documentation to separate the
three cases (absolute path, explicit relative path, other), and
also document that it raises LoadError on failure.
Fixes [Bug #16988] -
09:47 PM Feature #16978: Ruby should not use realpath for __FILE__
- > this code would change behavior:
Absolutely, different `$LOAD_PATH` must result in different behavior.
> ...
This is actually where I am coming from. I am not 100 % sure what is the spec of `require`, but I would expect that `req... -
09:18 PM Feature #16978: Ruby should not use realpath for __FILE__
- The "main file" (the file passed to `ruby myfile.rb` is also special in that `__FILE__` and `$0` can both be relative paths (basically the same path as passed on the command line).
-
- I don't think this is a bug. `__FILE__` is documented as follows: `The path to the current file.` Which path (real, absolute, relative, expanded) is not specified.
Not using the real path would lead to behavior that depends on the f... -
- The name `scan` seems confusing at least, since it has nothing to do with `String#scan`.
And `*_left` has no precedent in Ruby.
It seems it's basically recording the history of each block call (+ the initial value), maybe a name expr... -
08:34 PM Misc #17019: DevelopersMeeting20200720Japan
- * [Feature #17016] Enumerable#scan_left (parker)
* There is a [pull request](https://github.com/ruby/ruby/pull/3078) to implement `#scan_left`, i.e. the [scan](https://en.wikipedia.org/wiki/Prefix_sum#Scan_higher_order_function) opera... -
07:39 AM Misc #17019: DevelopersMeeting20200720Japan
- * [Feature #17000] 2.7.2 turns off deprecation warnings by default (mame)
* It is already decided that we fix, but it has been discussed how to fix. (Nagachika-san will not attend the meeting, but I'd like to hear opinions from commit... -
12:19 AM Misc #17019: DevelopersMeeting20200720Japan
- * [Feature #17004] Provide a way for methods to omit their return value (shyouhei)
* Is it a nice trick that we want to have, or a bad tool that is too easy to be abused? -
-
- Skip jit_test on some new RubyCI envs for now
Propagate JIT skip to all tests -
11:57 AM Bug #17021 (Closed): "arm64" and "arm" are mixed in RbConfig on Apple silicon
- When I checked the value of `RbConfig::CONFIG` on the Apple silicon macOS,e I found "arm64" and "arm" are mixed as following.
```
$ ruby -v
ruby 2.8.0dev (2020-07-09T09:14:53Z master addbb7d16e) [arm64-darwin20]
$ ruby -e 'requ... -
- disable to show the maximum number of threads.
On Deiban 9 environment, the thread tests failed and
this maximum threads information can finish up the machine
resources. To check it, I turned-off showing this inf... -
09:14 AM Revision addbb7d1 (git): Fix typos [ci skip]
-
08:11 AM Bug #17020: ObjectSpace.trace_object_allocations_stop raises if called before trace_object_allocations_start
- As explained in the PR, I have no idea how to write a regression test, because as long as `ObjectSpace.trace_object_allocations_start` was called at least once in the program, it no longer fails afterwards. That is what led me to think i...
-
- A possible alternative might be an exception, I thought first, but it isn’t a single condition.
Could you add the test? -
- How difficult it is to just disable some warnings!
I think that byroot is right here. Unfortunately, the practice of `$VERBOSE=true` is not so popular, so it will make many libraries "too verbose". Many unrelated warnings will makes... - 04:56 AM Revision cb0404cc (git): * 2020-07-09 [ci skip]
-
07/08/2020
-
07:42 PM Feature #16150: Add a way to request a frozen string from to_s
- I'd like to revisit this since it seems there's anecdotal evidence that returning frozen strings from Symbol#to_s is not as big a compatibility issue as thought.
Doing it for the next major release would make sense. -
- * [Feature #16923] Switch reserved for numbered parameter warning to SyntaxError (jeremyevans0)
* Is it OK to commit the patch?
* [Bug #9790] Zlib::GzipReader only decompressed the first of concatenated files (jeremyevans0)
* As I... -
- * [Feature #17018] Show cfunc frames in rb_profile_frames() (mame)
* My colleagues want this feature because it will make it easier to investigate an application bottleneck. -
- # The next dev meeting
**Date: 2020/07/20 13:00-17:00**
Place/Sign-up/Agenda/Log: https://github.com/ruby/dev-meeting-log/blob/master/DevelopersMeeting20200720Japan.md
- Dev meeting *IS NOT* a decision-making place. All decisions ... -
- sawa (Tsuyoshi Sawada) wrote in #note-1:
> What is wrong with the following?
> ...
Good question! Using `#inject` or `#each_with_object` cannot be done lazily. We need to pass a lazy enumerator *after* applying the fold-like operation.... -
04:00 AM Feature #17016: Enumerable#accumulate
- What is wrong with the following?
```ruby
[1, 2, 3].inject([0]){|a, e| a << a.last + e} # => [0, 1, 3, 6]
[1, 2, 3].each_with_object([0]){|e, a| a << a.last + e} # => [0, 1, 3, 6]
``` -
02:27 PM Revision 9fc25811 (git): Enhanced RDoc for Array (#3282)
- Methods:
reject!
reject
delete_if
zip
transpose
replace
clear -
- This patch will fix following build error on macOS 11.0 + Xcode 12 beta.
```
$ ./configure
$ make V=1
BASERUBY = /Users/watson/.rbenv/shims/ruby --disable=gems
CC = clang -fdeclspec
LD = ld
LDSHARED = cla... -
- The error is easy to reproduce:
e.g. on Ruby 2.3:
```
$ ruby -robjspace -e 'ObjectSpace.trace_object_allocations_stop'
-e:1:in `trace_object_allocations_stop': wrong argument type false (expected tracepoint) (TypeError)
from -e... -
- @nobu picked that fix in https://github.com/ruby/ruby/commit/6a826eb4b08484cb3b6279da905f5b9e03e0df5f
(details at https://github.com/ruby/spec/pull/778#issuecomment-655420419)
Thanks! -
-
-
- `rb_profile_frames()`, which is the backend of [stackprof](https://github.com/tmm1/stackprof), shows only ruby frames, so sometimes it is difficult to find a bottleneck in an application.
This will be gradually solved by "rubifying" all... -
08:43 AM Revision ff5e6603 (git): Added `NODE_SPECIAL_EXCESSIVE_COMMA` info to `ARGS` of `RubyVM::AbstractSyntaxTree`.
- 01:00 AM Revision 57cde28c (git): * 2020-07-08 [ci skip]
-
- ... because shifting by more than 31 bits has undefined behavior
(depending upon platform). Coverity Scan found this issue.
07/07/2020
-
08:56 PM Bug #17017: Range#max & Range#minmax incorrectly use Float end as max
- Add documentation change as an option
-
- Add full array example
-
- Add .to_a examples to clarify unexpected behaviour.
-
- While continuing to add edge cases to [`Range#minmax` specs](https://github.com/ruby/spec/pull/777), I discovered the following bug:
```ruby
(1..3.1).to_a == [1, 2, 3] # As expected
(1..3.1).to_a.max == 3 # As ex... -
- ## Proposal
UPDATE: Changed proposed method name from `#scan_left` to `#accumulate`.
Add an `#accumulate` method to `Enumerable`.
## Background
`#accumulate` is similar to `#inject`, but it accumulates the partial results tha... -
- ## Summary
`RubyVM::AbstractSyntaxTree.parse("proc { |a| }")` and `RubyVM::AbstractSyntaxTree.parse("proc { |a,| }")` have the same result.
Since `proc { |a| }` and `proc { |a,| }` have different meanings, I think we need a way to di... -
03:07 PM Feature #16986: Anonymous Struct literal
- Why is more special syntax needed, when it can just be a method?
```
def Kernel.AStruct(**key_values)
Struct.new(key_values.keys).new(key_values.values)
end
AStruct(a: 1, b: 2)
```
If that implementation isn't efficient en... -
- Well, I haven't heard from @akr. But if he doesn't respond in the next few days, I'll merge my pull request.
-
06:55 AM Bug #16922: single quote should be escaped in xml
- any update about this issue?
07/06/2020
-
09:18 PM Feature #17002 (Closed): Extend heap pages to exactly 16KiB
- Thanks, I committed it.
-
- This commit expands heap pages to be exactly 16KiB and eliminates the
`REQUIRED_SIZE_BY_MALLOC` constant.
I believe the goal of `REQUIRED_SIZE_BY_MALLOC` was to make the heap
pages consume some multiple of OS page size. 16KiB is conven... - 03:26 PM Revision b02a9584 (git): * 2020-07-07 [ci skip]
-
-
07:14 AM Revision 167d1394 (git): Inline builtin struct aref
- We don't do this for aset because it might raise a FrozenError.
```
$ benchmark-driver -v --rbenv 'before;after;before --jit;after --jit' benchmark/mjit_struct_aref.yml --repeat-count=4
before: ruby 2.8.0dev (2020-07-06T01:47:11Z master... -
07/05/2020
-
- Thanks! :)
-
-
- ## Summary
`RubyVM::AbstractSyntaxTree.parse("struct.field += foo")` has no operator.
I'm making a tool that uses AST, but I'm having trouble finding a way to distinguish `struct.field += foo` and `struct.field -= foo` because of thi... -
-
- The implementation of `Range#minmax` added in [d5c60214c45](https://github.com/ruby/ruby/commit/d5c60214c45bafc1cf2a516f852394986f9c84bb) causes the following incorrect behaviour:
```ruby
('a'...'c').minmax # => ["a", ["a", "b"]]
``... - 04:15 PM Revision efe851a0 (git): [ruby/bigdecimal] Version 2.0.1
- https://github.com/ruby/bigdecimal/commit/3fa4f2ac67
- 04:15 PM Revision 6a826eb4 (git): [ruby/bigdecimal] Return US-ASCII string from BigDecimal#to_s
- Fixes #159
https://github.com/ruby/bigdecimal/commit/57ee92e700 - 04:15 PM Revision e794d96c (git): [ruby/bigdecimal] Drop Ruby 2.3 support
- https://github.com/ruby/bigdecimal/commit/79a819d205
- 04:15 PM Revision 40b82afe (git): [ruby/bigdecimal] Support Ruby < 2.6
- https://github.com/ruby/bigdecimal/commit/61ec452599
- 04:15 PM Revision 03a33603 (git): [ruby/bigdecimal] Add Complex#to_d
- https://github.com/ruby/bigdecimal/commit/97e794ac97
-
- https://github.com/ruby/bigdecimal/commit/9d19e842ee
-
- This leaves the default definition, which will raise FrozenError.
https://github.com/ruby/bigdecimal/commit/05e843d838 - 04:15 PM Revision f00efef3 (git): [ruby/bigdecimal] Support a Complex in Kernel.BigDecimal()
- https://github.com/ruby/bigdecimal/commit/00795cb01f
-
- Both BigDecimal#clone and BigDecimal#dup return self, there is no
reason to have initialize_copy exposed as a Ruby method.
The same is true for initialize_clone and initialize_dup.
https://github.com/ruby/bigdecimal/commit/aaf237fa9e -
-
-
- 03:28 PM Revision 988b4bc9 (git): * 2020-07-06 [ci skip]
-
02:20 PM Bug #16787: [patch] allow Dir.home to work for non-login procs when $HOME not set
- Attaching two separate backport patches, one for branch 'ruby_2_6' and one for branch 'ruby_2_7'.
The changes for each patch were tested separately as outlined in the original issue description, and separate pull requests have been cr... -
-
-
- How should we proceed to include that fix in CRuby master?
-
- Actually, it's already been fixed upstream: https://github.com/ruby/bigdecimal/pull/160
Maybe the Ruby spec part of my PR is still useful. I'll submit it to https://github.com/ruby/spec -
- Sure.
-
- Could you submit this to https://github.com/ruby/bigdecimal?
-
- I submitted a PR for it: https://github.com/ruby/ruby/pull/3291
-
03:53 AM Bug #17011 (Closed): BigDecimal .to_s should not use ASCII-8BIT
- `BigDecimal#to_s` returns a `String` in ASCII-8BIT encoding. This is inconvenient in situations where the encoding determines program behaviour, since ASCII-8BIT is a special encoding that implies binary (non-textual) data, it can requir...
-
- jeremyevans0 (Jeremy Evans) wrote in #note-2:
> Please don't create issues just for backporting if there is an existing issue. Just update the Backport field in the original issue. I did this for #16787.
Thanks for the info, @jerem... -
-
-
-
- And c1463625555b061a2b94c3b6c5581730b482a285 too.
-
- This is for backport management.
The issue introduced bf1a6771f305ea286a3ae575676924551c03e857 for [Bug #15807] was fixed with bf1a6771f305ea286a3ae575676924551c03e857. -
-
-
01:44 AM Revision 95f5fd9a (git): fix up Primitive.cinit! code
- Recent changes break Primitive.cinit!(c_code) so fix it.
07/04/2020
-
- Fix CI failure like
http://ci.rvm.jp/results/trunk-mjit-wait@silicon-docker/3043247
introduced by a69dd699ee630dd1086627dbca15a218a8538b6f -
- configure: suppress icc warnings
Every time a pointer to/from VALUE conversion happens, these two
warnings are issued:
- warning #1684: conversion from pointer to same-sized integral type (potential portability ... -
-
- I created a pull request for this over on GitHub:
https://github.com/ruby/ruby/pull/3287
The integration tests are still running there as I write this. -
- The attached patch backports to ruby_2_6 change c15cddd1d515c5bd8dfe8fb2725e3f723aec63b8,
which fixes the issue described in bug #16787.
The original commit message:
Allow Dir.home to work for non-login procs when $HOME not se... -
- Please don't create issues just for backporting if there is an existing issue. Just update the Backport field in the original issue. I did this for #16787.
-
- I created a pull request for this over on GitHub:
https://github.com/ruby/ruby/pull/3288
The integration tests are still running as I write this.
-
- The attached patch backports to ruby_2_7 change c15cddd1d515c5bd8dfe8fb2725e3f723aec63b8,
which fixes the issue described in bug #16787.
The original commit message:
Allow Dir.home to work for non-login procs when $HOME not se... -
-
- FWIW I don't think it was "for Rails" but rather more use cases than that.
I want to note there is also both a performance cost and a semantics complication to only allowing Symbol keys.
If there is a call like `foo(**h)`, and `h` ha... -
- nagachika (Tomoyuki Chikanaga) wrote in #note-8:
> jeremyevans0 (Jeremy Evans) wrote in #note-7:
> ...
Attached is the patch. It's only a 1 line change to vm_args.c, but it includes all test changes. -
- jeremyevans0 (Jeremy Evans) wrote in #note-7:
> No, my proposal only affects keyword arguments. `rb_warn_check` is only called by the three functions that issue deprecation warnings for keyword arguments (all three callers are directly... - 05:02 PM Revision c4bdb642 (git): * 2020-07-05 [ci skip]
-
- The implementation of Range#minmax added in d5c60214c45 causes the
following incorrect behaviour:
('a'...'c').minmax => ["a", ["a", "b"]]
instead of
('a'...'c').minmax => ["a", "b"]
This is because the C implementation of Range#m... -
-
- Primitve.cexpr! and .cstmt! can access Ruby's parameter and
*local variables* (note that local parameters are also local
variables). However recent changes only allow to access
parameters. This patch fix it.
For example, the following c... -
- nobu (Nobuyoshi Nakada) wrote in #note-2:
> The description mentions the default value first, but the proposal will solve the default proc only.
Default value is a special (static) case of default proc, so I think having the latter w... -
- The description mentions the default value first, but the proposal will solve the default proc only.
-
04:41 AM Revision 7a5da7d5 (git): Allow hidden files with a hyphen to be source controlled
- Latest bundler includes a file named `.gitlab-ci.yml.tt`. Because of
this too specific .gitignore entry, it was failing to be properly
imported. -
- * Make Kernel#then, #yield_self, #frozen? builtin
* Fix test_jit -
- when an ISeq has multiple ivar accesses.
07/03/2020
-
08:29 PM Bug #17008: JIT enabled on Windows can cause constant conhost pop-ups
- k0kubun (Takashi Kokubun) wrote in #note-1:
> I have my own MinGW environment, but I've never observed such a behavior. Could you provide at least one way to reproduce the issue? I don't know of any GUI mingw32 application embedding Rub... -
- I have my own MinGW environment, but I've never observed such a behavior. Could you provide at least one way to reproduce the issue? I don't know of any GUI mingw32 application embedding Ruby 2.6+.
-
- I'm unsure of the min requirements for it to happen, but at least for ruby embedded in a gui mingw32 application, enabling JIT causes a conhost.exe popup for every gcc call, repeatedly stealing focus.
The easy fix is to add the CREATE_N... - 04:52 PM Revision ccb2e6ea (git): * 2020-07-04 [ci skip]
-
- * Rewrite Kernel#tap with Ruby
This was good for VM too, but of course my intention is to unblock JIT's inlining of a block over yield
(inlining invokeyield has not been committed though).
* Fix test_settracefunc
About the :tap deleti... -
- *Methods:
keep_if
delete
delete_at
slice! -
01:31 PM Feature #16986: Anonymous Struct literal
- the problem i have with that is that each time it creates a (cached) struct class.
after taking so long to make frozen literals and making even such frozen objects GC able,
something that creates new (class) objects which then can ne... -
10:52 AM Feature #16986: Anonymous Struct literal
- Hello, this is a great idea. I would just love to have this feature in Ruby, so I thought I may provide some helpful input:
First, I would like to read more ideas about the exact implementation, specially:
1. Will the literal pro... -
- ```ruby
class C
def foo
["C"]
end
end
refinement = Module.new do
R = refine C do
def foo
["R"] + super
end
include Module.new {
def foo
["M"] + super
end
}
end
en... -
- Sometimes, we need to create a hash that has explicit key-value pairs as well as a default value, but there has not been a way to do that at once. The most naive way is to do like this:
```ruby
h = {a: 1, b: 2}
h.default_proc = ->(h... -
- Use ID instead of GENTRY for gvars.
Global variables are compiled into GENTRY (a pointer to struct
rb_global_entry). This patch replace this GENTRY to ID and
make the code simple.
We need to search GENTRY from ID every time (st_lookup)... -
- * RUBY_DEBUG_LOG: Logging debug information mechanism
This feature provides a mechanism to store logging information
to a file, stderr or memory space with simple macros.
The following information will be stored.
* (1) __FILE__, __LI... -
06:16 AM Revision 01776ca1 (git): Ignore conftest.c under the ext directories for CodeQL analysis
-
- ```
/home/mame/work/ruby/.ext/common/json/common.rb:263: warning: Float 23456789012E666 out of range
``` - 02:27 AM Revision 05a12297 (git): * 2020-07-03 [ci skip]
-