Activity
From 08/26/2018 to 09/01/2018
09/01/2018
-
04:19 PM Bug #15039: Random.urandom and SecureRandom arc4random use
- shyouhei (Shyouhei Urabe) wrote:
> While I'm not against these points, I feel it is not a good idea to wait for such new API. Can we focus on fixing FreeBSD first? Maybe the OpenBSD issue should be handled separately in another thread... -
11:16 AM Bug #15039: Random.urandom and SecureRandom arc4random use
- Freaky (Thomas Hurst) wrote:
> jeremyevans0 (Jeremy Evans) wrote:
> ...
While I'm not against these points, I feel it is not a good idea to wait for such new API. Can we focus on fixing FreeBSD first? Maybe the OpenBSD issue should b... -
04:14 PM Bug #15027: When Struct#each method is overriden Struct#select and Struct#to_a use wrong collections
- Thank you for the code @nobu. I have applied Nobu's suggestion, please find updated patch in the attach.
> If you can override each, you can override other methods.
This is true, but doing this is very un-ruby-ish. Also, I don't t... -
01:22 PM Feature #13618: [PATCH] auto fiber schedule for rb_wait_for_single_fd and rb_waitpid
- > https://bugs.ruby-lang.org/issues/13618
Greg, do you think you can try this change for portability?
No rush, though, I will be mostly AFK and Queue/SizedQueue
changes will be trivial with the prep work so far...
Future chang... -
10:50 AM Revision cf85c2ef (git): bump up patchlevel for r64611
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_5@64612 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_5@64611 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
07:34 AM Revision 929e9713 (git): complex.c: simplify division result
- * complex.c (f_divide): canonicalize rationals to simplify integer
complex results.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64610 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
06:59 AM Revision 9563db5e (git): Re-try to add workaround for warnings
- ```
.../ext/psych/lib/psych/versions.rb:4: warning: already initialized constant Psych::VERSION
.../.ext/common/psych/versions.rb:4: warning: previous definition of VERSION was here
```
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@... -
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64608 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64607 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64606 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- ruby_2_5 r64605 merged revision(s) 64007,64019,64020.
-
- thread.c (do_select): fix leak on exception
When do_select is interrupted and raise happens from
RUBY_VM_CHECK_INTS_BLOCKING, the original FD sets we copied
do not get freed, leading to a memory leak. Wrap up al... -
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64604 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- ruby_2_5 r64603 merged revision(s) 64014.
-
- fix sum on infinity
* array.c (rb_ary_sum): consider non-finite floats.
[ruby-core:88024] [Bug #14926]
* enum.c (sum_iter): ditto.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_5@64603 b2dd03c8-... -
- ruby_2_5 r64602 merged revision(s) 63994.
-
- reduce tzset
* time.c (rb_localtime_r): call tzset() only after TZ environment
variable is changed.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_5@64602 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64601 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
01:32 AM Feature #14183: "Real" keyword argument
- jeremyevans0 (Jeremy Evans) wrote:
> Changing the callee side will not fix all cases. The `where` method supports more than just symbols keys in hashes. `where('table.id'=>1)` is supported, for example. Accepting a keyword args splat ...
08/31/2018
-
- jeremyevans0 (Jeremy Evans) wrote:
> As the maintainer of the OpenBSD ruby port, I'm requesting that ruby continue to use arc4random_buf(3) on OpenBSD. The OpenBSD getentropy(2) man page states:
> ...
I think there's a reasonable argu... -
06:56 PM Bug #15039: Random.urandom and SecureRandom arc4random use
- naruse (Yui NARUSE) wrote:
> I finally agree that
> ...
As the maintainer of the OpenBSD ruby port, I'm requesting that ruby continue to use arc4random_buf(3) on OpenBSD. The OpenBSD getentropy(2) man page states:
~~~
getentr... -
06:18 PM Bug #15039: Random.urandom and SecureRandom arc4random use
- Freaky (Thomas Hurst) wrote:
> naruse (Yui NARUSE) wrote:
> ...
As you mentioned, it's actually urandom.
I misunderstood.
> > > > But the size of those entropy is limited.
> ...
Yeah, it should block if there's no entropy.
> > ... -
- naruse (Yui NARUSE) wrote:
> Freaky (Thomas Hurst) wrote:
> ...
As far as I know getentropy(2) behaves as urandom, just with a size limit.
> > > But the size of those entropy is limited.
> ...
Yes, with the caveat that it should bl... -
- Freaky (Thomas Hurst) wrote:
> naruse (Yui NARUSE) wrote:
> ...
Even if /dev/random is not accessible, it can use getentropy(2).
> > But the size of those entropy is limited.
> ...
Yes.
But Random.urandom shouldn't be limited.
... -
05:38 PM Bug #14883 (Third Party's Issue): Ruby 2.5 Fails to Build on PowerPC 32-bit (BE)
- Issue with systemd containers, not with Ruby.
-
04:54 PM Misc #15053 (Closed): Using -fstack-protector-strong instead of -fstack-protector conditionally.
- When running a code analysis tool for ruby, I got a warning `Insufficient stack protection: -fstack-protector`, it seems setting "-fstack-protector-strong" is better than "-fstack-protector".
Currently in `configure.ac`, "-fstack-prot... -
- This reverts commit a5e5cfa3f650d4e78fb50e2df15c102ab56fca3c.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64600 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
- ```
.../ext/psych/lib/psych/versions.rb:4: warning: already initialized constant Psych::VERSION
.../.ext/common/psych/versions.rb:4: warning: previous definition of VERSION was here
```
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@... - 03:05 PM Revision 50cb77f2 (git): * 2018-09-01
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64598 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64597 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- mame (Yusuke Endoh) wrote:
> jeremyevans0 (Jeremy Evans) wrote:
> ...
Changing the callee side will not fix all cases. The `where` method supports more than just symbols keys in hashes. `where('table.id'=>1)` is supported, for example... -
- Here is a scenario where allowing "hash argument with omitted braces" causes a problem. Assume that we write a method "debug" which is equal to "Kernel#p".
```
def debug(*args)
args.each {|arg| puts arg.inspect }
end
```
Pas... -
- jeremyevans0 (Jeremy Evans) wrote:
> Having to change all calls from `where(:id=>1)` to `where({:id=>1})` makes the code uglier and is going to make most Ruby programmers less happy. Does this argument for explicitness lead to requirin... -
- mame (Yusuke Endoh) wrote:
> Jeremy, thank you for investigating the examples. I'd like to discuss this issue at the next developers' meeting.
> ...
I disagree. I migrated many applications and libraries from Ruby 1.8 to Ruby 1.9 (an... -
- Jeremy, thank you for investigating the examples. I'd like to discuss this issue at the next developers' meeting.
This is my personal current opinion: this change indeed requires users' action, however, I believe that the problem is ... -
- A keyword rest argument with empty hash, `**{}`, is removed during parsing phase. However, under the current spec of keyword parameters, this leads to a problem:
```
def foo(opt = "opt", **hsh)
p [opt, hsh]
end
foo({}, **{}) ... -
- * random.c (fill_random_bytes_syscall): use SecRandomCopyBytes in
Security framework on macOS 10.7 or later.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64596 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
08:54 AM Bug #14880: Time#localtime doesn't always seem to respect TZ
- @nobu That doesn’t seem to align with what the documentation says:
> Converts time to local time (using the local time zone in effect for this process) modifying the receiver.
-
- I think it should be fixed at the creation time.
-
- * [Feature #14183] "Real" keyword argument
* I'd like to hear matz's current opinion and other committers' ones. -
- Use `RB_INTEGER_TYPE_P()`.
-
07:22 AM Bug #15050: GC after forking with fibers crashes
- On 2018/08/31 15:14, Eric Wong wrote:
>> What is the problem [Bug #15041] want to solve?
>
> Switching fiber can crash in child process. r64589 test change
> shows it:
>
> ```
> - Fiber.new{ pid = fork {} }.resume
> + ... -
- Koichi Sasada <ko1@atdot.net> wrote:
> On 2018/08/31 12:50, Eric Wong wrote:
> > > [Bug #15041] hits something wrong?
> > > (sorry I needed to ask earlier)
> >
> > No, it's not wrong; but it is an incomplete fix. There
> > seem... -
- On 2018/08/31 12:50, Eric Wong wrote:
>> [Bug #15041] hits something wrong?
>> (sorry I needed to ask earlier)
>
> No, it's not wrong; but it is an incomplete fix. There
> seems to be several problems related to fork + Fiber
... -
- Koichi Sasada <ko1@atdot.net> wrote:
> On 2018/08/31 12:30, Eric Wong wrote:
> > Yes, fix one bug, hit another :< Story of my life
>
> [Bug #15041] hits something wrong?
> (sorry I needed to ask earlier)
No, it's not wrong; bu... -
- ko1@atdot.net wrote:
> Is this because [Bug #15041]?
Yes, fix one bug, hit another :< Story of my life
(I found these while working on auto-fiber) -
- Is this because [Bug #15041]?
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64595 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
08/30/2018
-
- To give an example of how much code this would break, let's use Redmine as an example, since it runs this bug tracker. For simplicity, let's limit our analysis to the use of a single method, ActiveRecord's `where` method. ActiveRecord'...
-
08:11 PM Feature #15049: [Request] Easily access all keyword arguments within a method
- shevegen (Robert A. Heiler) wrote:
> I have had a somewhat related idea in that I wanted a way to access all
> ...
Tbh I hadn't thought too much about naming... Was just trying to get something mocked up quickly. Either way, glad to s... -
07:44 PM Feature #15049: [Request] Easily access all keyword arguments within a method
- I have had a somewhat related idea in that I wanted a way to access all
parameters in a simple, short manner. A bit similar to $1 $2 etc... for
regexes, just that we could do so programmatically for the arguments
passed into a method ... -
- As a developer, I'd like to leverage the power of keyword arguments (requirements, defaults, etc) and then be able to access a hash of all the arguments supplied.
~~~ ruby
def foo(bar:, baz: 1, qux: 2...)
post('/', body: kargs)
e... -
- normalperson@yhbt.net wrote:
> https://bugs.ruby-lang.org/issues/15050
This fixes the immediate bug, but I think there's other problems
with stack management
```
diff --git a/cont.c b/cont.c
index 7bc1724176..ab42dfb27a 100644... -
- Not sure when to work on this, so leaving this here for now...
The management of stacks for root fiber and regular
fibers is different and confusing. Perhaps unifying
thread and fiber stack cache is the best way to go.
Is a sepa... - 07:14 PM Revision cec21388 (git): * 2018-08-31
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64594 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 07:14 PM Revision 13f49235 (git): cont.c (rb_fiber_atfork): th->root_fiber may not exist
- Otherwise, bootstraptest/test_fork.rb fails with -DVM_CHECK_MODE=2
[Bug #15041]
Fixes: r64589 "cont.c: set th->root_fiber to current fiber at fork"
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64593 b2dd03c8-39d4-4d8f-98ff-823fe6... -
05:09 PM Misc #15007: Let all Init_xxx and extension APIs frequently called from init code paths be considered cold
- normalperson (Eric Wong) wrote:
> lourens@bearmetal.eu wrote:
> ...
Yes, explains behaviour observed through smaps - correct, `MAP_HUGETLB` works. Will clean it up during the week and submit as a new issue. -
- lourens@bearmetal.eu wrote:
> * `MADV_HUGEPAGE` - fallback option, implicit request, kernel
> will map it implicit if not aligned to hugepage boundary, or
> right away if aligned to hugepage boundary (I implemented with
> the 2MB al... -
- ko1 (Koichi Sasada) wrote:
> I agree to introduce COLD attribute.
> ...
Somewhat off topic, but fits with the general direction here.
I have a proof of concept going with mapping the text section to hugepages after a few evenings of... -
01:48 PM Feature #15047: Documentation and more functions for Hash functions in C API
- Can I submit the documentation for this as a GitHub PR? I just to make updates to README.EXT.
Also, thank you for the help I will make a note of that in my patch. -
12:57 PM Feature #15047: Documentation and more functions for Hash functions in C API
- > Also, including more functionality like getting the size a Hash directly through
> ...
Use RHASH_SIZE macro. -
- More documentation would always be very helpful. It makes "entry" into the
ruby world easier when there is good documentation. Perhaps after ruby 3x3
we could add a new meme - four times as good documentation. :D -
- Currently the C API supports various Array functions like `rb_ary_new()` and String
functions like `rb_str_new2()` I could hardly find any documentation in the README.EXT
for working with hashes. Instead, I relied on this blog: http://... -
- naruse (Yui NARUSE) wrote:
> Freaky (Thomas Hurst) wrote:
> ...
... but you've just described concerns over situations where such APIs are unavailable. If you have no /dev/urandom why would you have a /dev/random? If you have no non-... -
- naruse (Yui NARUSE) wrote:
> > > On Darwin, there seems similar issue as OpenBSD.
> ...
As far as I investigated, it's implemented with CCRandomCopyBytes() and fork safe. We should use this.
https://stackoverflow.com/a/21736905/515572 -
- Freaky (Thomas Hurst) wrote:
> naruse (Yui NARUSE) wrote:
> ...
There's /dev/random and getentropy.
But the size of those entropy is limited.
> > And as Jeremy already pointend, OpenBSD doesn't have urandom alternative syscall beca... -
- I'm now for the addition. By the way, don't you need `Array#intersection`?
-
08:51 AM Feature #14097: Add union and difference to Array
- As @mame already said, I gave a talk at Euruko about this issue:
https://youtu.be/jUc8InwoA-E?t=2m54s
I did an standing up voting. As in the video you can not see the audience, here is the picture I took during the talk with the re... -
- * [Feature #14097] Add union and difference methods to Array (@ana06)
* Addition of two new methods in aim of readability, ease of use, efficiency and consistence. There are already PR for both methods.
* @matz told me to add this ... -
- On 2018/08/29 17:09, Eric Wong wrote:
> Shows me: `resume': fiber called across stack rewinding barrier (FiberError)
Yes, I tried similar code.
Not sure why...
Nobu?
--
// SASADA Koichi at atdot dot net - 03:24 AM Revision 3bcb4ff8 (git): hrtime.h: fix typo in non-builtin overflow check
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64592 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
08/29/2018
-
- naruse (Yui NARUSE) wrote:
> I agree with your fundamental concept: if there's more code, there's more bugs.
> ...
If the OS can't provide entropy to a process through urandom or an equivalent syscall, how is it going to safely seed a ... -
- I agree with your fundamental concept: if there's more code, there's more bugs.
I don't fully agree with "userspace CSPRNG is harmful" because it's required.
For example, ruby can't read /dev/urandom if it runs in jail.
And as Jer... -
- shyouhei (Shyouhei Urabe) wrote:
> Freaky (Thomas Hurst) wrote:
> ...
Doesn't the name rather waggle its eyebrows at you while glancing pointedly at /dev/urandom? Why else call it that?
> My attempt when fixing #9569 was that arc4r... -
- jeremyevans0 (Jeremy Evans) wrote:
> Freaky (Thomas Hurst) wrote:
> ...
Even if that were the case, it's storing its state in userspace, it's permuting that state in userspace. Where it comes from is beside the point, as is how often ... -
- Freaky (Thomas Hurst) wrote:
> > Or is it not about the code quality in general? Maybe the problem here is that having _any_ bits of instructions in a userland?
> ...
Hmm, OK. I respect your feeling that the arc4random() in FreeBSD (... -
- Freaky (Thomas Hurst) wrote:
> Would rand() stop being a userspace PRNG if it automatically called srand() every 200,000 calls?
If it received the argument to srand() from the kernel, then maybe. :)
> ...
I think in general Secure... -
- shyouhei (Shyouhei Urabe) wrote:
> Frankly, I'm not sure how less trustworthy FreeBSD libc is, compared to its kernel. They are developed by the same people in a same repository.
FreeBSD /dev/(u)random's seen a *lot* more attention ... -
- jeremyevans0 (Jeremy Evans) wrote:
> Freaky (Thomas Hurst) wrote:
> ...
Would rand() stop being a userspace PRNG if it automatically called srand() every 200,000 calls?
> arc4random_buf(3) is basically like fread(3), where getentrop... -
- Frankly, I'm not sure how less trustworthy FreeBSD libc is, compared to its kernel. They are developed by the same people in a same repository. Or is it not about the code quality in general? Maybe the problem here is that having _any...
-
- Freaky (Thomas Hurst) wrote:
> jeremyevans0 (Jeremy Evans) wrote:
> ...
srand/rand is the classic userspace PRNG, where all future output is based only on the initial input.
arc4random_buf(3) is basically like fread(3), where getent... -
- jeremyevans0 (Jeremy Evans) wrote:
> On OpenBSD, arc4random is not really a userspace PRNG in the sense that it is reseeded on a regular basis using new entropy data from the kernel
It runs code in userspace to stretch a small amou... -
09:05 PM Bug #15043: MinGW intermittent test failure - TestIO#test_select_leak
- @normalperson Eric,
Thanks for looking at it and also the patch. It passed test-all, but as mentioned, it's intermittent and I can't repo it with ruby-loco... Greg -
- Greg.mpls@gmail.com wrote:
> Bug #15043: MinGW intermittent test failure - TestIO#test_select_leak
> https://bugs.ruby-lang.org/issues/15043
Thanks for the report. r64590 ought to fix it -
- Applied in changeset trunk|r64590.
----------
test/ruby/test_io.rb (test_select_leak): use handle_interrupt
Interrupt timing is tricky and it's possible the target
thread is still stopped from the previous loop iteration.
[ruby-core:8... -
- @normalperson Eric,
Thought I'd ping you on this, as the test is very new. The above test is not failing on Travis Ubuntu, nor Appveyor mswin or ruby-loco.
It is intermittently failing on the recently added Ruby Appveyor mingw j... - 07:47 PM Revision 5ba404f4 (git): * 2018-08-30
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64591 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 07:47 PM Revision f1119509 (git): test/ruby/test_io.rb (test_select_leak): use handle_interrupt
- Interrupt timing is tricky and it's possible the target
thread is still stopped from the previous loop iteration.
[ruby-core:88732] [Bug #15043]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64590 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
- If I put this into a .rb file:
puts 'secret'.encoding
ENV['PASS'] = 'secret'
puts ENV['PASS'].encoding
On my system I get these two Strings output:
UTF-8
ISO-8859-1
My environment is, aka my current loc... -
02:44 PM Bug #15044 (Closed): ENV encoding not UTF-8 by default
- ~~~
$ irb
2.5.1 :001 > 'secret'.encoding
=> #<Encoding:UTF-8>
2.5.1 :002 > ENV['PASS'] = 'secret'; ENV['PASS'].encoding
=> #<Encoding:US-ASCII>
2.5.1 :009 > ENV['PASS'] = 'Ł'
=> "\u0141"
2.5.1 :010 > ENV['PASS'].encoding
=> ... -
04:16 PM Bug #15045 (Closed): Test suite fails/hangs with stronger cryptography enforcement
- Testing in Fedora Rawhide with ```$ sudo update-crypto-policies --set NEXT``` [1] which sets a higher bar for cryptography algorithms, the test suite fails, several test cases timeouts or even hangs (the WEBrick test suite in particular ...
-
- ```diff
diff --git a/struct.c b/struct.c
index 7de46980aa..e4c875b5be 100644
--- a/struct.c
+++ b/struct.c
@@ -860,6 +860,9 @@ rb_struct_inspect(VALUE s)
static VALUE
rb_struct_to_a(VALUE s)
{
+ if (!rb_method_basic_defini... -
11:22 AM Feature #14975: String#append without changing receiver's encoding
- ioquatix (Samuel Williams) wrote:
> ```ruby
> ...
Didn't think about it, sorry. But it's only pseudocode. The non-block `#force_encoding` will definitely not work on frozen strings, but the version with a block could be made to work be... -
- ```ruby
b = 'a'.force_encoding('BINARY')
u = "\u00ff".force_encoding('UTF-8') # aside: force_encoding here is a no-op,
# because \u automatically produces UTF-8
u.force_encoding('BINARY') do
... -
- As mentioned, the general idea of Ruby m17n is that strings that only contain ASCII bytes (8th bits are all 0) are treated as US-ASCII and can be combined with any ASCII-compatible encoding (taking on that encoding as a result).
The p... -
- gotoken (Kentaro Goto) wrote:
> For example, `"ss"` and `"st"` are mapped `"ß"` (`"\u00DF"`) and `"st"` (`"\uFB06"`).
> ...
I still have to verify this, but currently I strongly suspect that the problem is NOT in SpecialCasing, but in... -
- Eric Wong wrote:
> It will be difficult/unsafe to use auto-fiber/Thread::Coro
> [Feature #13618] without this
I think I can get around this by making iom a GC root,
so all auto-yielded Fibers get marked. -
- Koichi Sasada wrote:
> Bug #595: Fiber ignores ensure clause
> http://redmine.ruby-lang.org/issues/595
What's the status of this? (I don't understand Japanese)
It will be difficult/unsafe to use auto-fiber/Thread::Coro
[Featur... -
- ko1@atdot.net wrote:
> (I'm not sure why fork removes other fibers, but)
fib = Fiber.new { puts " HI" }
fork { fib.resume }
Process.waitall
Shows me: `resume': fiber called across stack rewinding barrier (FiberError)
If w... -
- Applied in changeset trunk|r64589.
----------
cont.c: set th->root_fiber to current fiber at fork
Otherwise, th->root_fiber can point to an invalid Fiber,
because Fibers do not live across fork. So consider
whatever Fiber is running t... -
- (I'm not sure why fork removes other fibers, but)
I think it is okay. let's try.
-
- cont.c: set th->root_fiber to current fiber at fork
Otherwise, th->root_fiber can point to an invalid Fiber,
because Fibers do not live across fork. So consider
whatever Fiber is running the root fiber.
- 08:04 AM Revision ac543abe (git): cont.c: set th->root_fiber to current fiber at fork
- Otherwise, th->root_fiber can point to an invalid Fiber,
because Fibers do not live across fork. So consider
whatever Fiber is running the root fiber.
[ruby-core:88723] [Bug #15041]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64... - 02:23 AM Revision 43d3f813 (git): * 2018-08-29
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64588 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 02:23 AM Revision 0b44744d (git): cont.c (rb_context_t): remove ensure_list
- It is unused (we use rb_execution_context_t.ensure_list instead)
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64587 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
08/28/2018
-
- On OpenBSD, arc4random is not really a userspace PRNG in the sense that it is reseeded on a regular basis using new entropy data from the kernel (arc4random_buf(3) -> _rs_random_buf -> _rs_stir_if_needed -> _rs_stir -> getentropy(2)). Se...
-
- Random.urandom defaults to arc4random() on a lot of platforms, including FreeBSD.
On all currently released versions of FreeBSD, arc4random() is, as the name suggests, a dubious ARC4-based userspace PRNG dating from circa 1997. Given... -
- I just noticed that the Appveyor MinGW build is not showing the installed bundled gems in the console output, although the mswin build is.
See r64585 test, [mswin](https://ci.appveyor.com/project/ruby/ruby/build/9494/job/ml60euv5uxa84... -
02:55 PM Revision 8867f285 (git): Tag a couple specs failing on AIX
- * The rest seems OS bugs, MRI bugs or incomplete IPv6 implementation:
https://rubyci.org/logs/rubyci.s3.amazonaws.com/aix71_ppc/ruby-trunk/log/20180828T103302Z.fail.html.gz
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64586 b2dd0... -
01:42 PM Revision 8a999649 (git): Fixed installation failure with mswin environment.
- [ruby-core:88699][Bug #15035]
This patch was provided by MSP-Greg.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64585 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64584 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- [@Student](https://bugs.ruby-lang.org/users/6774) as I just answered in the [PR](https://github.com/ruby/ruby/pull/1747#issuecomment-416494207) about the same concern, I'll answer here to:
> I think there are some uses of case. First,... -
08:19 AM Feature #15036 (Closed): after upgrade when run apt upgrade metasploit
-
- Applied in changeset trunk|r64583.
----------
rest parameter optimization [Feature #15010]
* vm_args.c: rb_ary_dup(args->rest) to be used at most once during
parameter setup. [Feature #15010]
A patch by chopraanmol1 (Anmol Chopra) ... -
- It seems fine.
I'll commit it. -
- * vm_args.c: rb_ary_dup(args->rest) to be used at most once during
parameter setup. [Feature #15010]
A patch by chopraanmol1 (Anmol Chopra) <chopraanmol1@gmail.com>.
* array.c (rb_ary_behead): added to remove first n elements.
git... -
- Eregon (Benoit Daloze) wrote:
> There are certainly cases where a C extension wants to check the argument is an Integer, and its range doesn't matter, isn't it?
For example?
-
- @MSP-Greg
Thanks for your patches. I will merge a patch of `tool/rbinstall.rb` after confirming with my windows environment. -
- @hsbt
Thanks for applying the patch. I should have been clearer, as the patch fixes two issues, both involving RubyGems, but one of the patched files (tool/rbinstall.rb) is patched for the parameter passed to a RubyGems method. That... -
- Applied in changeset trunk|r64582.
----------
Fixed test failures in mswin environment at r64555.
[ruby-core:88699][Bug #15035]
This patch was provided by MSP-Greg. -
- @hsbt
Windows has been failing on install, seems due to a combination of RubyGems changes.
ruby-loco is passing with the patch, and the ruby/ruby mswin is completing make install (but failing on some recent spec changes). See GH P... -
- [ruby-core:88699][Bug #15035]
This patch was provided by MSP-Greg.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64582 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
- On 2018/08/27 18:17, Eric Wong wrote:
>>> A joke; at least as far as we know it is a joke :>
>>
>> I'm not sure why uint64_t doesn't solve time machine problem and uint128_t
>> solves it. Any RFC arguing it? (time machines should no... -
- I hope this is relevant.
I found an interesting article here: http://www.oilshell.org/blog/2016/12/11.html
It describes the process Python uses.
It looks like it's standardised in a way that might be a useful goal for `RubyVM::A... - 12:24 AM Revision 4d9c2377 (git): thread_pthread.h: rename `gvl.acquired' to `gvl.owner' and document
- `acquired' was an old boolean variable, but nowadays it is a
rb_thread_t pointer; "gvl.owner" seems like a more appropriate
name. And document the contended path including waitq, timer,
and timer_err.
git-svn-id: svn+ssh://ci.ruby-lan... -
- Thank you for your report.
This is a issue tracker for ruby development, and I'm not at all familiar with Yast2 package manager. I just searched by the error message and found:
https://lists.opensuse.org/opensuse-factory/2018-08/m...
08/27/2018
- 11:39 PM Revision fcaf8745 (git): thread_pthread.c: document sigwait_th and sigwait_fd [ci skip]
- This is an important concept to document, I think.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64580 b2dd03c8-39d4-4d8f-98ff-823fe69b080e - 11:29 PM Revision 1b1274f7 (git): thread_pthread.c: fix deadlock on test_thread.rb::test_signal_at_join
- Fixes: r64575 ("avoid lock ping-pong in do_gvl_timer & ubf_select")
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64579 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
09:37 PM Bug #15034 (Closed): Crash in Opensuse Software management
- Hi,
the problem appears when starting the Software Management component of Open Suse's Yast2 package manager.
The software management component starts up for some while, then crashes. The problem has appeared several times in the pa... -
09:19 PM Bug #15033 (Closed): Encoding fallback uses wrong character when multiple conversions are required
- When converting a string from one encoding to another that involves multiple conversions, the proc passed to `encode` will be called with the incorrect value if the conversion fails in the middle of the conversion.
For example,
```... -
- > https://bugs.ruby-lang.org/issues/13618
Btw, many of my recent changes ([Misc #15014] (rb_hrtime_t)),
and process.c/thread*.c cleanups in r64542, r64575, r64576,
r64577 are preparatory work for this feature that were
beneficial ... -
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64578 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 06:37 PM Revision e61e6ae5 (git): process.c: fix potential missed wakeups in r64576
- Oddly, all existing test cases passed multiple times
before this patch (even with --jit-wait), so this seems
like a difficult failure to prove.
Fixes: r64576 ("process.c: simplify SIGCHLD-based waitpid")
git-svn-id: svn+ssh://ci.ruby-l... -
06:34 PM Feature #14912: Introduce pattern matching syntax
- "The patterns are run in sequence until the first one that matches."
This means O(n) time for matching. If we are adding this with changes to compiler, we should try to compile it to hash lookup O(1) time
Does this allow nesting of... - 05:17 PM Revision 9d4027b5 (git): process.c: simplify SIGCHLD-based waitpid
- Introduce a new rb_thread_sleep_interruptible that does not
execute interrupts before sleeping. Skipping the interrupt
check before sleep is required for out-of-GVL ruby_waitpid_all
to function properly when setting waitpid_state.ret
N... - 05:17 PM Revision 943bf37b (git): thread_pthread.c: avoid lock ping-pong in do_gvl_timer & ubf_select
- This simplifies the locking logic somewhat.
While we're at it, designate_timer_thread is worthless in
ubf_select because gvl_acquire_common already guarantees there
is a gvl.timer if gvl->waitq is populated.
In the future (for auto-fib... - 04:38 PM Revision cda13d8c (git): * 2018-08-28
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64574 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64573 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 04:22 PM Revision 293ed16e (git): * append newline at EOF.
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64572 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64571 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
04:07 PM Bug #14920: Backport r63857 to fix performance problem on Time class in MacOs systems
- Hi @usa.
63994 alone doesn't fix the issue. I just cloned ruby_2_4 with r64560 and run the same benchmark script and got the same result. Only r63857 fix the issue. Can you backport r63857 too?
```
"Main Process: 0.100000 0.0... -
01:59 PM Bug #14920: Backport r63857 to fix performance problem on Time class in MacOs systems
- ruby_2_4 r64560 merged revision(s) 63994.
- 02:49 PM Revision 58689fa7 (git): * append newline at EOF.
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64570 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64569 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64568 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- * To be able to run spec/ruby/command_line/dash_encoding_spec.rb
with the in-repo build.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64567 b2dd03c8-39d4-4d8f-98ff-823fe69b080e - 02:25 PM Revision cea7d047 (git): * append newline at EOF.
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64566 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64565 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- ruby_2_4 r64564 merged revision(s) 64092.
- 02:18 PM Revision cccbcd17 (git): merge revision(s) 64092: [Backport #14942]
- win32.c: limit write size on console
* win32/win32.c (constat_parse): split long buffer and limit write
size on a console, as well as rb_w32_write.
[ruby-dev:50597] [Bug #14942]
git-svn-id: svn+ssh://ci.ruby... -
- ruby_2_4 r64563 merged revision(s) 64071.
- 02:15 PM Revision 94e65989 (git): merge revision(s) 64071: [Backport #14941]
- ruby.c: taint ARGV on Windows
* ruby.c (external_str_new_cstr): strings come from the external
should be tainted. [ruby-dev:50596] [Bug #14941]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_4@64563 b2dd... -
- ruby_2_4 r64562 merged revision(s) 64014.
- 02:11 PM Revision ba757763 (git): merge revision(s) 64014: [Backport #14926]
- fix sum on infinity
* array.c (rb_ary_sum): consider non-finite floats.
[ruby-core:88024] [Bug #14926]
* enum.c (sum_iter): ditto.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_4@64562 b2dd03c8-... -
- ruby_2_4 r64561 merged revision(s) 64007,64019,64020.
- 02:08 PM Revision 95abe79e (git): merge revision(s) 64007,64019,64020: [Backport #14929]
- thread.c (do_select): fix leak on exception
When do_select is interrupted and raise happens from
RUBY_VM_CHECK_INTS_BLOCKING, the original FD sets we copied
do not get freed, leading to a memory leak. Wrap up al... - 01:59 PM Revision 1bbf7789 (git): merge revision(s) 63994: [Backport #14920]
- reduce tzset
* time.c (rb_localtime_r): call tzset() only after TZ environment
variable is changed.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_4@64560 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
- ruby_2_4 r64559 merged revision(s) 63696.
- 01:48 PM Revision 4941bba3 (git): merge revision(s) 63696: [Backport #14853]
- variable.c: fix receiver on private constant
* variable.c (rb_const_search): fix NameError :receiver attribute
on private constant, should raise with the included module, not
the ICLASS.
git-svn-id: svn+ssh:... -
- ruby_2_4 r64558 merged revision(s) 62731,62735.
- 01:47 PM Revision 7d957281 (git): merge revision(s) 62731,62735: [Backport #14495]
- Bug Fix Enumerator::Lazy#uniq state for multiple call
* enumerator.c (lazy_uniq_i): create new hash for each calls.
[Fix GH-1820]
Currently
2.5.0-preview1 :001 > arr = (0..100).lazy.uniq{|i| i... -
- eregontp@gmail.com wrote:
> There are certainly cases where a C extension wants to check the argument is an Integer, and its range doesn't matter, isn't it?
> ...
Not currently, just two calls to `RB_TYPE_P` but that's a trivial
stati... -
- There are certainly cases where a C extension wants to check the argument is an `Integer`, and its range doesn't matter, isn't it?
Is there a macro to check for that easily? -
- +1 for Hanmac.
C extension writers should check `T_FIXNUM` or `T_BIGNUM` explicitly.
-
05:49 AM Feature #15030: Have T_INTEGER constant for checking Integer types in C API
- hm that is not that easy, while `Fixnum` and `Bignum` are deprecated on the ruby side,
on the C side they still exist
thats why a single `T_INTEGER` check might not work that way you want -
- After Integer unification in Ruby 2.4, I think it makes sense to introduce a T_INTEGER
type that helps in directly testing if a Ruby object is an Integer in C API.
For example, currently we need to write `RB_TYPE_P(obj, T_FIXNUM)` f... -
- ko1 (Koichi Sasada) wrote:
> `rb_obj_is_range()` can be accepted like `rb_obj_is_proc` and so on.
> ...
Sure :) -
- `rb_obj_is_range()` can be accepted like `rb_obj_is_proc` and so on.
`T_xxx` shows data layout of `RVALUE`.
-
- Range is an in-built type of Ruby and it would be better for C extension
writers if there existed a constant `T_RANGE` that could be directly used
for checking whether a `VALUE` is a `Range`.
Currently we need to write `CLASS_OF(obj... -
- > Is there a use-case for using this new method with non-binary strings?
Yes, predictably appending strings without changing receiver encoding. If I make a buffer with a specific encoding, I'd prefer not to have it change encoding wit... -
- I agree the current behavior for appending with binary strings is surprising.
Especially since the result of the operation depends on whether the receiver and argument are 7-bit or not.
However, it makes sense to me to change the rec... -
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64557 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
10:08 AM Feature #15010: Reduce allocation for rest parameters
- @ko1, I've added new field rest_dupped to args_info.
Updated patch https://bugs.ruby-lang.org/attachments/7344/Reduce-allocation-for-rest-parameters-v1.patch
-
- ko1 (Koichi Sasada) wrote:
> # Idea (as my understanding)
> ...
Yes, and once a rest parameter is duped it mutates the array in case if rest_index is modified (Previously, only args_setup_post_parameters used to mutate rest parameter).... -
- Sorry for late response.
# Idea (as my understanding)
~a rest parameter" is dup multiple times because of current implementation. Only 1 "dup" is needed. They should be eliminate.
The patch try to manage "dup'ed or not" by passing... -
- @ko1, It would be great if you could review https://bugs.ruby-lang.org/attachments/7343/Reduce-allocation-for-rest-parameters-v1.patch
- 10:05 AM Revision 73c8b1ea (git): * remove trailing spaces.
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64556 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- * It's preparation to release RubyGems 3.0.0.beta2 and Ruby 2.6.0
preview 3.
* https://github.com/rubygems/rubygems/compare/v3.0.0.beta1...fad2eb15a282b19dfcb4b48bc95b8b39ebb4511f
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@... -
- Koichi Sasada <ko1@atdot.net> wrote:
> On 2018/08/27 16:16, Eric Wong wrote:
> > Yep. Maybe some small systems can benefit with
> > usec/msec resolution and uint32_t, though.
>
> My comment is, if it assumes uint64_t, it is worth... -
- On 2018/08/27 16:16, Eric Wong wrote:
> Yep. Maybe some small systems can benefit with
> usec/msec resolution and uint32_t, though.
My comment is, if it assumes uint64_t, it is worth to write it
explicitly (I verified this years... -
- ko1@atdot.net wrote:
> Thanks!
>
> > + * Hi-res monotonic clock. It is currently nsec resolution, which has over 500 years of range (unsigned).
>
> uint64?
You're welcome. Yep. Maybe some small systems can benefit with
use... -
- Thanks!
> + * Hi-res monotonic clock. It is currently nsec resolution, which has over 500 years of range (unsigned).
uint64?
> ...
what is this?
- 08:48 AM Revision 3a83ba90 (git): hrtime.h: add note explaining current use of uint64_t [ci skip]
- [ruby-core:88678]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64554 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
06:31 AM Bug #13671: Regexp with lookbehind and case-insensitivity raises RegexpError only on strings with certain characters
- Thanks shyouhei for your pointing out.
I imagine another Rexexp option, say `//I`, which is almost the same as `//i` except for never-applying SpecialCasing mapping.
This change extends Unicode matching indeed but does not introduc... -
- gotoken (Kentaro Goto) wrote:
> By the way, this expansion by `//i` option looks over kill for me.
> ...
I know how you feel. Too bad we are just doing what Unicode specifies to do.
See also http://unicode.org/faq/casemap_charpro... -
- Thanks znz. The workaround is helpful. And I understood what was happened.
https://github.com/k-takata/Onigmo/issues/92#issuecomment-373981492 shows how some combinations of letters are variable length.
For example, `"ss"` and `"st"`... -
- You can use `(?:s)` instead of `s` for workaround.
```
$ ruby -ve '/(?<=ast)/iu'
ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-darwin17]
-e:1: invalid pattern in look-behind: /(?<=ast)/i
-e:1: warning: possibly useless use of ... -
- I encountered a non `ss` case. Is this a same problem?
```
% ruby -ve '"".match(/(?<=ast)/ui)'
ruby 2.6.0dev (2018-08-27 trunk 64549) [x86_64-linux]
-e:1: invalid pattern in look-behind: /(?<=ast)/i
```
It was reproduced in ve... -
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64553 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64552 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64551 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 05:32 AM Revision 49a55f28 (git): thread.c: check interrupts explicitly in select/ppoll blocking regions
- The logic around blocking_region_begin is confusing to me,
but the goal of this patch is to ensure rb_sigwait_fd_get
and rb_sigwait_fd_put are matched.
In other words, we don't want a thread to hold sigwait_fd
forever if an exception is... -
- ioquatix (Samuel Williams) wrote:
> Doing a binary comparison would be acceptable.
I don't think so. As Akira already pointed out,
- Addrinfo have "padding" bits which are not stable. Binary comparison might fail at that point.
... -
- Doing a binary comparison would be acceptable.
-
- @nobu Isn't changing TZ enough that `#localtime` should do something?
-
- @marcandre I already added support for coverage of templates and it works pretty well. You need to be careful with template code generation, but for the most part it's fairly straight forward. Thanks for your detailed reply - if we could...
-
- Did you take a look at https://github.com/ioquatix/covered - I'd be interested in your feedback. Can we discuss further, and also how to improve performance? There is some more discussion here: https://bugs.ruby-lang.org/issues/14888
-
- What about using trace points?
-
- * [Feature #14975] String#append without changing receiver's encoding (ioquatix)
* Can we review this PR? We need to decide functionality (i.e. not changing receivers encoding, rejecting encoding if it's not same, unless receiver is b... - 01:30 AM Revision bab586e7 (git): * properties.
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64549 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 01:30 AM Revision 6a052fcd (git): * remove trailing spaces.
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64548 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- * https://github.com/ruby/rdoc/compare/v6.0.4...v6.1.0.beta1
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64547 b2dd03c8-39d4-4d8f-98ff-823fe69b080e - 12:44 AM Revision 41fb2436 (git): * properties.
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64546 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 12:44 AM Revision 56ac079e (git): * 2018-08-27
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64545 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
- * Update bundled libyaml-0.2.1 from 0.1.7.
https://github.com/ruby/psych/pull/368
* Unify Psych's API: To use keyword arguments with method call.
https://github.com/ruby/psych/pull/358
git-svn-id: svn+ssh://ci.ruby-lang.org/ru...
08/26/2018
-
- naruse (Yui NARUSE) wrote:
> methodmissing (Lourens Naudé) wrote:
> ...
Thanks for clarifying the context there. I looked @ MSVC attributes / annotations and `cold` (or anything similar) is not supported, thus for `cold` and `hot` attr... - 12:41 PM Revision e0311eb8 (git): test/ruby/test_io.rb (test_recycled_fd_close): Linux workaround
- Arch Linux CI still seems to timeout on this test...
Note, I can't reproduce the failures in these tests on a
FreeBSD 11.1 VM while infinite-looping, even without the
"th.join(0.001)". It doesn't seem related to the use of
rb_wait_for_... - 12:41 PM Revision 70d6c1a1 (git): thread_sync.c: common wakeup_{one,all} implementation
- This let us avoid looping in rb_szqueue_max_set, saves us
some lines of code and reduces binary size slightly
(numbers from 32-bit x86):
text data bss dec hex filename
before: 91272 392 156 91820 ... -
- as a middle way, can't we just do the "is overwritten by user" check?
i think i have seen it on other classes like Array or Hash
so it checks if the `each` method is overwritten, in that case, call the `Enumerable` method,
if not, c... -
- Since matz asked for feedback, just a comment - I have no particular pro or con opinion
per se, mostly because I very rarely use the Struct/OpenStruct family; I usually just end
up writing a "real" class instead and adapt it to what is... -
- I'm against changing the current behavior. If you can override `each`, you can override other methods. If we wanted to be consistent about making this change, we would probably need to remove most the following additional core methods ...
-
01:56 AM Bug #15027 (Feedback): When Struct#each method is overriden Struct#select and Struct#to_a use wrong collections
- The proposed behavior is more consistent but slower. I am not sure it's a good idea to hinder performance by supporting consistency in the rare case. Any opinion?
Matz.
-
- Thank you for the answer. I'm assigning this ticket to matz.
-
- @mame
> However, I'm now getting confused. I thought you agreed that Array#union should be non-destructive, but in your talk you explained it was a destructive union operator. What are you driving at?
In the talk I wanted to focus ... -
- 04:29 AM Revision 84859fd2 (git): process.c: remove worthless waitpid_sys macro
- It is identical to do_waitpid, and the win32 version will not
be needed for MJIT (since win32 does not suffer from the
waitpid(-1, ...) conflict where waits can get stolen.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64541 b2dd03c...