Activity
From 04/20/2018 to 04/26/2018
04/26/2018
-
10:49 PM Bug #14607 (Closed): Fix use of the rb_profile_frames start parameter
- Applied in changeset trunk|r63265.
----------
Fix use of `rb_profile_frames` start parameter
rb_profile_frames was always behaving as if the value given for the
start parameter was 0.
The reason for this was that it would check if (st... - 10:49 PM Revision 707cbe6f (git): * 2018-04-27
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63266 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
10:49 PM Revision d676ad10 (git): Fix use of `rb_profile_frames` start parameter
- rb_profile_frames was always behaving as if the value given for the
start parameter was 0.
The reason for this was that it would check if (start > 0) { then
continue without updating the control frame pointer or anything other
than decr... -
09:04 PM Feature #12607: Ruby needs an atomic integer
- I don't like pulling in the entire concurrent-ruby gem only for a few lines of code. I've implemented my own slow but thread-safe Counter here:
https://github.com/mperham/sidekiq/blob/b58c505df3501d8c1de5207552d68dd2d9abea31/lib/side... -
12:37 PM Feature #14715 (Open): Pathname#== should compare by #realpath instead of #to_s
- Hi Core Team,
I'd like to report a confusing behaviour I noticed with Pathname#== comparison method.
See the following example:
Supposed I've created a file "file.txt" on /home/user/ directory.
~~~ ruby
require 'pathname'
... -
12:36 PM Revision 81f02142 (git): win32/Makefile.sub: LIBDIR_BASENAME
- * mjit.c (init_header_filename): support LIBDIR_BASENAME.
* win32/Makefile.sub (config.h): ditto.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63264 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
12:36 PM Revision 8036af48 (git): ruby.c (ruby_init_loadpath_safe): constify
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63263 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
11:51 AM Bug #14714: Ruby 2.5.1 Segmentation Fault in GC
- As it occurs in GC, it may reproduce with a few test by setting `GC.stress` to `true`.
-
11:02 AM Bug #14714: Ruby 2.5.1 Segmentation Fault in GC
- I have reverted to ruby 2.4.4 and done rspec/spec/features and the problem does not occur, to this increases the probability that it is a ruby 2.5.1 problem.
-
10:28 AM Bug #14714 (Closed): Ruby 2.5.1 Segmentation Fault in GC
- I am using Ruby 2.5.1, and running ruby on rails 5.1.4 with rspec-rails 3.7.2 and capybara 3.0.2.
When I run my feature tests, i.e. rspec spec/features, I get a segmentation fault. When I found the test in which the fault is occurri... -
11:05 AM Bug #11779: Module#using does not make sense as a method
- From RSpec:
~~~ ruby
let(:test_class) do
Class.new(described_class) do
private
using SomeRefiningModule # => RuntimeError: Module#using is not permitted in methods
def foo
end
end
end
~~~
-
09:32 AM Bug #14713: <internal:prelude>:132:in `__write_nonblock': Protocol wrong type for socket (Errno::EPROTOTYPE)
- ioquatix (Samuel Williams) wrote:
> Very occasionally on macOS, using Ruby 2.5.0, I get the above error.
It happens often on macOS.
> ...
Probably, but not sure.
-
05:00 AM Bug #14713 (Closed): <internal:prelude>:132:in `__write_nonblock': Protocol wrong type for socket (Errno::EPROTOTYPE)
- Very occasionally on macOS, using Ruby 2.5.0, I get the above error.
```
Traceback (most recent call last):
23: from /Users/samuel/.rvm/gems/ruby-2.5.0/gems/async-1.6.0/lib/async/task.rb:74:in `block in initialize'
22: from /User... -
09:30 AM Feature #14701: If the object is not frozen, I want to be able to redefine the compound assignment operator.
- I think it is enough to change the meaning of
~~~ ruby
x += a
~~~
to
~~~ ruby
if x.respond_to?(:'+=')
x.send(:'+=',a)
else
x = x + a
end
~~~
Matz rejected this in http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/... -
09:28 AM Feature #14701: If the object is not frozen, I want to be able to redefine the compound assignment operator.
- nobu (Nobuyoshi Nakada) wrote:
> It seems a different story from `+=`.
I am sorry for getting off the point, but I wanted to say that `add!` method does not solve the problem.
> ...
This cannot be a solution because both inplace a... -
08:56 AM Feature #14701: If the object is not frozen, I want to be able to redefine the compound assignment operator.
- masa16 (Masahiro Tanaka) wrote:
> Former NArray had `add!` method, but it results in a confused result.
It seems a different story from `+=`.
> ...
It feels that `a[0,true]` should return `inplace` object.
Otherwise `+=` would no... -
07:21 AM Feature #14701: If the object is not frozen, I want to be able to redefine the compound assignment operator.
- As first thing to say, we do not expect changing behavior of `+=` for ruby built-in types such as string or array.
We want to have an extension point to redefine `+=` for our library such as NArray.
How it should behave is the design... -
06:03 AM Feature #14701: If the object is not frozen, I want to be able to redefine the compound assignment operator.
- I agree with matz and eregon.
Honestly, I first thought that it was a good idea to split `+` and `+=`. But by investigating Python, I now think that it easily leads to strange behavior. One example is what eregon and sonots said:
... -
04:24 AM Feature #14701: If the object is not frozen, I want to be able to redefine the compound assignment operator.
- > Most notably, it means other variables pointing to the same NArray would get modified in place by +=, which seems highly unexpected.
It seems expected and natural for me. It behaves similarly with other destructive methods of ruby:
... -
03:58 AM Feature #14701: If the object is not frozen, I want to be able to redefine the compound assignment operator.
- What about
```ruby
na = Numo::Int32[5, 6]
a = na.inplace
a += 1
``` -
01:17 AM Feature #14701: If the object is not frozen, I want to be able to redefine the compound assignment operator.
- Former NArray had `add!` method, but it results in a confused result.
~~~ ruby
require 'narray'
a = NArray.int(2,2).indgen!
a[1,true] += 10
p a
# => NArray.int(2,2):
# [ [ 0, 11 ],
# [ 2, 13 ] ]
a = NArray.int(2,... -
06:04 AM Feature #13618: [PATCH] auto fiber schedule for rb_wait_for_single_fd and rb_waitpid
- samuel@oriontransfer.org wrote:
> If you are unsure of a good definition for the reactor
> pattern, I think this is a good one:
> https://en.wikipedia.org/wiki/Reactor_pattern except the
> assumption that you need to invert flow con... -
04:57 AM Feature #13618: [PATCH] auto fiber schedule for rb_wait_for_single_fd and rb_waitpid
- If you are unsure of a good definition for the reactor pattern, I think this is a good one: https://en.wikipedia.org/wiki/Reactor_pattern except the assumption that you need to invert flow control which is not necessary using fibers.
... -
12:12 AM Bug #14712 (Closed): test_step does not work for implementations with full 64-bit fixnum
- Applied in changeset trunk|r63261.
----------
test_numeric.rb: loose precision assertion
* test/ruby/test_numeric.rb (TestNumeric#test_step): remove a
loose precision assertion, as Float cannot keep complete
precision. [ruby-core:... - 12:12 AM Revision 4ac77028 (git): * 2018-04-26
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63262 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
12:12 AM Revision b7d260f1 (git): test_numeric.rb: loose precision assertion
- * test/ruby/test_numeric.rb (TestNumeric#test_step): remove a
loose precision assertion, as Float cannot keep complete
precision. [ruby-core:86684] [Bug #14712]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63261 b2dd03c8-39d4-...
04/25/2018
-
07:45 PM Feature #14709: Proper pattern matching
- On considering a bit more, I've come up with a few syntax variants for such a feature.
Preference towards avoiding `%p` and other shorthands in favor of a more self-descriptive word like `matches` or `match`.
### 1. matches
Ther... -
05:22 AM Feature #14709: Proper pattern matching
- ## On Pattern Matching in Other Languages
For continuity, we should also go over implementations in other languages.
Note that I am not an expert in any of these languages, and only proficient in Javascript. I would encourage other... -
02:53 AM Feature #14709: Proper pattern matching
- It should also be mentioned that the way I achieved the pattern matching mentioned above was by using a status container in the style of Elixir (`{:ok, result}` except `[true, result]`):
https://github.com/baweaver/qo/blob/83577f80a47... -
01:35 AM Feature #14709: Proper pattern matching
- It was mentioned to me on Reddit that I should detail exactly what cases of pattern matching would entail.
For the case of Qo, these include four primary categories:
* Array matches Array
* Array matches Object
* Hash matches Has... -
12:39 AM Feature #14709: Proper pattern matching
- I am the author of aforementioned Qo, so I believe I can shed some light on a few things:
* The thinking behind Qo
* Reusable syntax features
* Making it feel like Ruby
* Shortcomings of current syntax
* Performance considerations... -
05:35 PM Bug #14712 (Closed): test_step does not work for implementations with full 64-bit fixnum
- In test/ruby/test_numeric.rb, in test_step, there's the following logic:
```ruby
assert_operator((0.0).step(bignum.to_f, 1.0).size, :>=, bignum) # may loose precision
```
where bignum is
```
bignum = RbConfig::LIMITS['FIXNUM_... -
04:53 PM Feature #14701: If the object is not frozen, I want to be able to redefine the compound assignment operator.
- I agree with matz.
Python's inplace operators sound like a fundamental design flaw to me,
because `a += b` is no longer `a = a + b` but sometimes something completely different and there is no way to differentiate at the source level... -
03:22 PM Feature #14701: If the object is not frozen, I want to be able to redefine the compound assignment operator.
- ANOTHER IDEA:
How about allowing to redefine `+!` instead of `+=` although it looks not intuitive for me, but it would be a ruby way. -
03:19 PM Feature #14701: If the object is not frozen, I want to be able to redefine the compound assignment operator.
- > Use append instead of += for arrays.
`+=` operation for NArray is totally different with `append` for ruby array.
What we want to do with `+=` operation for NArray is to do element-wise addition like
```
narray([1, 2, 3]) += na... -
02:00 PM Revision 6509c178 (git): [DOC] Fix capitallizing [ci skip]
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63260 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
11:57 AM Bug #14711 (Closed): URI::HTML5ASCIIINCOMPAT returns expression
- Applied in changeset trunk|r63259.
----------
common.rb: unused constant
* lib/uri/common.rb (URI::HTML5ASCIIINCOMPAT): remove the constant
which has been unused since r40460, and wrong since r49069 due
to the operator precedence. ... -
07:00 AM Bug #14711 (Closed): URI::HTML5ASCIIINCOMPAT returns expression
- Since Ruby 2.3.0 URI::HTML5ASCIIINCOMPAT returns expression instead of an array.
I guess it's a backward compatibility issue.
```
➜ ~ RBENV_VERSION=2.2.5 ruby -e "require 'uri'; p URI::HTML5ASCIIINCOMPAT"
[#<Encoding:UTF-7 (dum... -
11:56 AM Revision 27e97285 (git): common.rb: unused constant
- * lib/uri/common.rb (URI::HTML5ASCIIINCOMPAT): remove the constant
which has been unused since r40460, and wrong since r49069 due
to the operator precedence. [ruby-core:86678] [Bug #14711]
git-svn-id: svn+ssh://ci.ruby-lang.org/rub... -
10:43 AM Bug #12305: "can't remove str from str_id" at unregister_sym
- Was there any resolution to this issue? We are encountering the same issue now that we have upgraded our Ruby from 2.1.6 to 2.3.3. This happens randomly throughout our framework when we are trying to run regression.
Any and all infor... -
08:16 AM Revision 512041bf (git): Avoid "should_not raise_error" in Thread#raise spec
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63258 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
08:15 AM Revision 9498d66c (git): Fix style in Thread#raise spec
- * Add space after { and before }.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63257 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
08:11 AM Revision 66058dd5 (git): Use Thread.pass in loop{} to check interrupts more often
- * The spec now runs in ~5ms vs ~100ms before.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63256 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
08:03 AM Misc #14698: DevelopersMeeting20180517Japan
- > please follow the format to make easy to copy&paste.
@ko1 I've updated my comment with a list of "functional programming" tickets. -
01:54 AM Misc #14698: DevelopersMeeting20180517Japan
- Hi,
I will copy & paste topics to ticket's "description" later so that please follow the format to make easy to copy&paste.
mrkn's exmaple is perfect:
```
* [Feature #14697] Introducing Range#% as an alias to Range#step (mrkn)
... -
01:47 AM Misc #14698: DevelopersMeeting20180517Japan
- I would also like to propose that this be discussed as it recently came up: https://bugs.ruby-lang.org/issues/14709
Though it may be premature. -
06:45 AM Bug #14708 (Closed): argument stack underflow (-1) (SyntaxError)
- Applied in changeset trunk|r63255.
----------
compile.c: fix unconditional branch optimization
* compile.c (iseq_peephole_optimize): add dummy `putnil` after a
`jump` replacing an unconditional branch, to adjust removed
`dup`. [ru... -
06:45 AM Revision 067066fc (git): compile.c: fix unconditional branch optimization
- * compile.c (iseq_peephole_optimize): add dummy `putnil` after a
`jump` replacing an unconditional branch, to adjust removed
`dup`. [ruby-core:86666] [Bug #14708]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63255 b2dd03c8-39d...
04/24/2018
-
11:28 PM Feature #14710 (Open): I'd like to know from C API that "It has only one reference to Ruby object" to determine whether it is a temporary object.
- I'd like to know from C API that "It has only one reference to Ruby object" to determine whether it is a temporary object.
Because broadcasting with python numpy is faster for
(1 line case)
y = x + 1 + 1
than
(2 ... -
08:32 PM Feature #14705 (Closed): [PATCH] eval.c (ruby_setup): disable THP on Linux
- Applied in changeset trunk|r63253.
----------
eval.c (ruby_setup): disable THP on Linux
Transparent Huge Pages (THP) decrease the effectiveness of
CoW-friendly GC because it decreases page granularity. That is,
a forked process dirtyi... - 08:32 PM Revision b768dc22 (git): * 2018-04-25
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63254 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 08:31 PM Revision acbbf8b0 (git): eval.c (ruby_setup): disable THP on Linux
- Transparent Huge Pages (THP) decrease the effectiveness of
CoW-friendly GC because it decreases page granularity. That is,
a forked process dirtying one bit of CoW-shared memory can
trigger a copy of a huge page (2MB on x86-64) instead ... -
07:35 PM Bug #13524: miniruby: [BUG] Segmentation fault at 0x0055e487e00230 ruby 2.4.1p111 (2017-03-22 revision 58053) [x86_64-li
- All, I modified Shyouhei's Dockerfile to use alpine3.7 and jemalloc 5.0.1 and it no longer segfaults:
~~~
FROM ruby:2.4.4-alpine3.7
~~~
~~~
Step 8/10 : ADD segv.c /tmp/segv.c
---> be86245150aa
Step 9/10 : RUN gcc -ljemall... -
04:19 PM Feature #14709: Proper pattern matching
- I think only matz can answer that.
On a side note, I am not aware of a issue request about it. Has this
already been suggested, or is it only a presentation? If it has not
yet been proposed formally through an issue request on the t... -
03:19 PM Feature #14709 (Closed): Proper pattern matching
- On RubyKaigi 2017, there was a [presentation](http://rubykaigi.org/2017/presentations/yotii23.html) of Yuki Torii about possible implementation of pattern matching.
The syntax proposed in presentation was:
```ruby
res = [:ng, 500]... -
02:45 PM Bug #14708 (Closed): argument stack underflow (-1) (SyntaxError)
- ~~~
ruby -e '[].each { false || [].each { } }' # OK
ruby -e '[].each { nil.nil? || [].each { } }' # OK
ruby -e '[].each { true || [].each { } }'
-- raw disasm--------
<L498253648>
trace: 100
0000 nop ... -
12:40 PM Feature #13683: Add strict Enumerable#single
- How about `Enumerable#just(num=1)` or `Enumerable#only(num=1)`?
-
12:26 PM Bug #14707 (Closed): String#scan(/\K/) has changed its behavior in ruby 2.5
- Applied in changeset trunk|r63252.
----------
string.c: fix scanned substring with `\K`
* string.c (scan_once): fix the matched substring with `\K`, the
beginning of that string may differ from the matched position.
[ruby-core:8666... -
11:48 AM Bug #14707 (Closed): String#scan(/\K/) has changed its behavior in ruby 2.5
- It seems `\K` does not work as expected any more:
```
% ruby -ve 'p "a1 a2 a3".scan(/a\K./)'
ruby 2.3.7p456 (2018-03-28 revision 63014) [x86_64-darwin17]
["1", "2", "3"]
ruby 2.4.5p297 (2018-03-29 revision 63036) [x86_64-darwi... -
12:25 PM Revision 2c8f16e6 (git): string.c: fix scanned substring with `\K`
- * string.c (scan_once): fix the matched substring with `\K`, the
beginning of that string may differ from the matched position.
[ruby-core:86663] [Bug #14707]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63252 b2dd03c8-39d4-4d8... -
12:22 PM Feature #12607: Ruby needs an atomic integer
- Should this issue be assigned to Matz?
-
09:53 AM Feature #14706: Atomic Integer incr/decr
- I guess this may come up several times in the future. Would
this be a good candidate for discussion in the upcoming
ruby developer meeting? I don't want to suggest it myself
since I was not the one who started the thread (that was
...
04/23/2018
-
11:46 PM Revision 3471d0f6 (git): rescue Errno::EPROTOTYPE
- * test/webrick/test_httpserver.rb (test_gigantic_request_header):
Errno::EPROTOTYPE is sometimes raised on Mac OS X 10.10.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63251 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
09:40 PM Feature #14706: Atomic Integer incr/decr
- Jeremy, good point. I would agree that the different semantics are enough to justify a different class. Namespacing it within the Thread class also seems reasonable, e.g. `Thread::Counter` so it would be available if you `require 'thre...
-
09:27 PM Feature #14706: Atomic Integer incr/decr
- Related ticket with some discussion already (and partially rejected) https://bugs.ruby-lang.org/issues/12607
-
09:18 PM Feature #14706: Atomic Integer incr/decr
- I think this feature in general is a good candidate for addition to stdlib.
I don't think the `Integer` class is appropriate for this, because (some) Integers are immediate values, and all Integers are frozen, and you can't modify the... -
09:08 PM Feature #14706: Atomic Integer incr/decr
- I've always missed that feature as well, so I'm +1 to this. This and many other built-in thread-safe helpers by the way ;)
-
08:53 PM Feature #14706 (Closed): Atomic Integer incr/decr
- Ruby does not any thread-safe way to implement simple counters without a Mutex. Today Ruby provides Integer#succ but this funcalls "+", making it thread-unsafe as far as I know.
I'd propose adding Integer#incr(amount=1) and Integer#r... - 04:20 PM Revision 6c4cab00 (git): * 2018-04-24
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63250 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
04:20 PM Revision f84efbaa (git): revert r63212
- except test_jit.rb.
In some situations, this generates a wrong code. I'll add a test for it
later but let me revert this to make it work for now.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63249 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
01:56 PM Feature #14697: Introducing Range#% as an alias to Range#step
- I'm supposing that this new notation of Range#step is mostly used for slicing numerical arrays like Numo::NArray.
This usecase is very similar to Python's slice notation used for slicing numpy's arrays.
With this new notation, `ary[:... -
12:51 PM Revision 4cae5353 (git): compile.c: copy a short insn with leave
- * compile.c (iseq_peephole_optimize): copy not only `leave`, with
a non-operand instruction, which are not longer than `jump`.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63248 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
12:09 PM Revision 425118b8 (git): [DOC] URI::Generic#port returns Integer [ci skip]
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63247 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
09:52 AM Revision c6ad7a62 (git): compile.c: insn before pop
- * compile.c (iseq_peephole_optimize): more eliminatable
instructions before `pop` without side effects.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63246 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
07:02 AM Misc #14698: DevelopersMeeting20180517Japan
- zverok (Victor Shepelev) wrote:
> > your favorite ticket numbers you want to ask to discuss with your SHORT comment or summary.
> ...
I would second this list, especially #6284 (composition of Procs) and #13581 (syntax sugar for method... -
07:00 AM Feature #13581: Syntax sugar for method reference
- sevos (Artur Roszczyk) wrote:
> After a while I am becoming a bigger fan of the triple colon operator. We could implement a class MethodSelector for handling the logic and the operator would be expected to return an instance of the clas... - 05:54 AM Revision 87ca4f30 (git): test/ruby/test_io.rb: add extra Thread#join to delay close
- Maybe this fixes some CI failures. Also, use different timeouts
for each item for hopefully easier diagnosis.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63245 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
02:07 AM Revision 1222560a (git): thread_pthread.c: fallback to CLOCK_REALTIME
- * thread_pthread.c (Init_native_thread): fallback to the default
CLOCK_REALTIME when failed to set to CLOCK_MONOTONIC, e.g. on
Solaris. [Misc #14497]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63244 b2dd03c8-39d4-4d8f-98ff-8...
04/22/2018
-
10:33 PM Misc #12004: Code of Conduct
- gabe@binti.com wrote:
> Issue #12004 has been updated by gkop (Gabe Kopley).
>
> matz (Yukihiro Matsumoto) wrote:
>> We have set our Code of Conduct.
>>
>> https://www.ruby-lang.org/en/conduct/
>>
>> I hope it works. We may upg... -
10:21 PM Feature #14705 (Closed): [PATCH] eval.c (ruby_setup): disable THP on Linux
- Transparent Huge Pages (THP) decrease the effectiveness of
CoW-friendly GC because it decreases page granularity. That is,
a forked process dirtying one bit of CoW-shared memory can
trigger a copy of a huge page (2MB on x86-64) inste... -
09:21 PM Feature #14703 (Closed): [PATCH] net/imap: set SO_KEEPALIVE on TCP sockets
- Applied in changeset trunk|r63243.
----------
net/imap: set SO_KEEPALIVE on TCP sockets
Otherwise connections (commonly on IDLE, but it could be any
command) may never receive notifications of link errors.
[ruby-core:86628] [Feature #... - 09:21 PM Revision c4056a4a (git): net/imap: set SO_KEEPALIVE on TCP sockets
- Otherwise connections (commonly on IDLE, but it could be any
command) may never receive notifications of link errors.
[ruby-core:86628] [Feature #14703]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63243 b2dd03c8-39d4-4d8f-98ff-82... -
07:41 PM Revision 75e5a91e (git): sprintf.c: fix typo
- * sprintf.c: [DOC] fix typo.
Patch by Lazarus Lazaridis (iridakos). [Fix GH-1789]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63242 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
04:33 PM Revision 532464d1 (git): Same as the last commit (comment out tests for CI).
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63241 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 04:03 PM Revision fdf2764e (git): * 2018-04-23
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63240 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
04:03 PM Revision dd45691d (git): Skip some tests to make CI healthy.
- r63236 (or r63237) introduces test failures and CI shows errors.
This commit makes skipping these tests. Please revert this commit
after tests (and rubyspec) work fine.
Failure log example:
https://gist.github.com/ko1/8456cf25fe35a696b... -
12:09 PM Misc #14497 (Closed): [PATCH] thread*: all condvars are monotonic
- Applied in changeset trunk|r63238.
----------
thread*: all condvars are monotonic
There's no reason to use CLOCK_REALTIME for any condvars in Ruby.
Indeed, we initialized all condvars with RB_CONDATTR_CLOCK_MONOTONIC
anyway; so simplif... - 12:09 PM Revision ed590bdb (git): thread*: all condvars are monotonic
- There's no reason to use CLOCK_REALTIME for any condvars in Ruby.
Indeed, we initialized all condvars with RB_CONDATTR_CLOCK_MONOTONIC
anyway; so simplify our code and reduce ifdefs.
[ruby-core:85639] [Misc #14497]
git-svn-id: svn+ssh:... -
09:43 AM Bug #14600: test-all & REXML - fixup
- I've enabled almost tests.
The tests reveals some XPath related bugs. They are good tests.
I've fixed most of the XPath related bugs. Namespace axis related XPath isn't fixed yet. I'll work on it later. - 09:38 AM Revision 7a6f3410 (git): rexml: Fix XPath bug of //#{ELEMENT_NAME}[#{POSITION}]
- The position should be counted for each nodeset but the previous
implementation counts position for union-ed nodeset.
For example, "/a/*/*[1]" should be matched to "<c1/>" and "<c2/>" with
the following XML.
<a>
<b>
<... - 08:09 AM Revision 4d15e619 (git): rexml: Fix XPath bug of /#{ELEMENT_NAME}
- It doesn't mean that all elements which name "ELEMENT_NAME" with any
namespace URI including null namespace URI. It means that all elements
which name "ELEMENT_NAME" with null namespace URI.
https://www.w3.org/TR/1999/REC-xpath-19991116... -
05:53 AM Misc #14692: Question: Ruby stdlib's Option Parser
- nobu (Nobuyoshi Nakada) wrote:
> Currently, it is not able.
> ...
Can you please show me an example code on how to achieve this ?
I see the issue is in `parser_in_order` method
```
begin
sw, = co... -
04:01 AM Revision b252d8ad (git): mjit_compile.c: comment the intention of r63092 [ci skip]
- It's for "leave" instruction.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63235 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
12:47 AM Revision 56e87b2c (git): made *.cmd excutable
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63234 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
12:46 AM Revision 04ac0363 (git): made *.cmd excutable
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63233 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
12:12 AM Bug #14681: `syswrite': stream closed in another thread (IOError)
- samuel@oriontransfer.org wrote:
> > We no longer set O_NONBLOCK on sockets/pipes by default since
> > 1.9+; and but that didn't help with slow filesystems whose
> > buffers are full (or using weird stuff like O_SYNC/O_DIRECT).
> C... -
12:02 AM Revision 0b429ad2 (git): Makefile.in: MJIT_ARCHFLAG
- * Makefile.in (mjit_config.h): separate MJIT_ARCHFLAG for each
architecture on universal binary. cannot use precompiled-header
with multiple -arch options.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63232 b2dd03c8-39d4-4d8f-...
04/21/2018
- 11:32 PM Revision fed7f81b (git): test/ruby/test_io.rb: try to diagnose stuck test_recycled_fd_close
- I can't reproduce the problem myself, but gets loop seems ought
to give more useful information for tracking down where we're
stuck, at least.
Followup-to: r63217
cf. http://ci.rvm.jp/results/trunk-test@frontier/804284
git-svn-id: svn... -
11:02 PM Revision c83decf6 (git): mjit.c: check pch status
- * mjit.c (mjit_add_iseq_to_process, mjit_get_iseq_func): check if
pch failed before timedout.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63230 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
10:12 PM Bug #14681: `syswrite': stream closed in another thread (IOError)
- > We no longer set O_NONBLOCK on sockets/pipes by default since
> ...
Can I set this to avoid the breaking code path? I know the write would succeed every time in this case.
> signaled cross-thread IO#close
Do you mind explaining ... -
06:12 PM Bug #14681: `syswrite': stream closed in another thread (IOError)
- samuel@oriontransfer.org wrote:
> I reviewed your suggestion, and while it (in theory) works
> with the original example, it won't work with my actual use
> case which uses are set of shared threads to implement
> background workers... -
06:03 PM Bug #14681: `syswrite': stream closed in another thread (IOError)
- samuel@oriontransfer.org wrote:
> Excuse my ignorance, but if you call write, why can't you just
> directly invoke `::write`? Why do you need to do
> `rb_thread_io_blocking_region`?
rb_thread_io_blocking_region releases the GVL b... -
01:26 PM Bug #14681: `syswrite': stream closed in another thread (IOError)
- Excuse my ignorance, but if you call write, why can't you just directly invoke `::write`? Why do you need to do `rb_thread_io_blocking_region`?
-
01:18 PM Bug #14681: `syswrite': stream closed in another thread (IOError)
- I reviewed your suggestion, and while it (in theory) works with the original example, it won't work with my actual use case. I have a fixed number of worker threads reading from a shared job queue and executing jobs - when the job is com...
-
01:14 PM Bug #14681: `syswrite': stream closed in another thread (IOError)
- Eric, thanks so much for your commitment to fixing this issue and for taking a look at my specific use case.
I will try out your suggestions. If it works, it's good enough.
I look forward to your future bug fix.
-
09:04 AM Bug #14681: `syswrite': stream closed in another thread (IOError)
- Eric Wong <normalperson@yhbt.net> wrote:
> https://80x24.org/spew/20180421021502.31552-1-e@80x24.org/
> Note: the /* TODO: check func() */ in rb_thread_io_blocking_region
> But that WIP patch is broken...
>
> I think w... -
03:32 AM Bug #14681: `syswrite': stream closed in another thread (IOError)
- samuel@oriontransfer.org wrote:
> Bug #14681: `syswrite': stream closed in another thread (IOError)
> https://bugs.ruby-lang.org/issues/14681
There's two bugs, here, I think. I made r63216 because it
became obvious to me with the... -
12:33 AM Bug #14681: `syswrite': stream closed in another thread (IOError)
- Thanks, fixing now.
- 08:04 PM Revision 83bd262f (git): * 2018-04-22
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63229 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
08:04 PM Revision 300b22dc (git): Improve docs for URI library
- * lib/uri/generic.rb: [DOC] fix invalid example code to make it
syntax highlighted; drop unnecessary `puts', `p'; adapt to current
inspect format without Object id; do not display unnecessary return
values in examples; fix or preve... -
07:50 PM Misc #12004: Code of Conduct
- matz (Yukihiro Matsumoto) wrote:
> We have set our Code of Conduct.
> ...
Let's upgrade it, Matz. -
01:15 PM Revision c04881f9 (git): test_jit.rb: follow the change of instruction
- in r63225. Not strictly needed but to avoid confusion.
The JIT compiler itself seems working fine.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63227 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
01:09 PM Bug #14561: Consistent 2.5.0 seg fault in GC, related to accessing an enumerator in a thread
- Thanks so much for your effort to isolate this issue.
I don't think any of my code is violating "First, Fiber.new and Fiber#resume must be in same thread, but Enumerator.new and Enumerator#peek don't have to be."
I will check. -
11:33 AM Feature #13618: [PATCH] auto fiber schedule for rb_wait_for_single_fd and rb_waitpid
- samuel@oriontransfer.org wrote:
> It would be also be pretty awesome if you could actually
> supply a reactor to use, e.g. `Fiber.new(io_reactor:
> my_reactor)`. In this case, blocking operations would call
> `my_reactor.wait_readab... -
11:12 AM Feature #14624: #{nil} allocates a fresh empty string each time
- nobu@ruby-lang.org wrote:
> https://github.com/nobu/ruby/tree/feature/opt_to_s
Btw, I also had [ruby-core:81905] [Feature #13715] from last
year but forgot about it :x -
10:52 AM Revision fde115b1 (git): compile.c: optimize checktype
- * compile.c (optimize_checktype): optimize `checktype` instruction
on a literal.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63226 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
10:52 AM Revision a3fe1034 (git): insns.def: checktype
- * insns.def (checktype): split branchiftype to checktype and
branchif, to make branch condition negation possible.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63225 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
07:48 AM Revision c458aeff (git): compile.c: renamed macro arguments
- * compile.c (INSERT_BEFORE_INSN, INSERT_BEFORE_INSN1): rename
argument `prev` as `next`.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63224 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
06:52 AM Bug #14659: segfault in ConditionVariable#broadcast and ConditionVariable#signal
- nbeyer@gmail.com wrote:
> Bug #14659: segfault in ConditionVariable#broadcast and ConditionVariable#signal
> https://bugs.ruby-lang.org/issues/14659
I suspect this is identical to https://bugs.ruby-lang.org/issues/14634
Can you ... - 06:45 AM Revision ee29985d (git): rexml: Enable more tests
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63223 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 06:43 AM Revision da89931d (git): rexml: Fix XPath concat() implementation
- * lib/rexml/functions.rb (REXML::Functions.concat): Implement.
* test/rexml/test_jaxen.rb: Enable one more test.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63222 b2dd03c8-39d4-4d8f-98ff-823fe69b080e - 06:39 AM Revision 6793c0a2 (git): rexml: Fix XPath string() implementation
- * lib/rexml/functions.rb( REXML::Functions.string):
* Support context node.
* Fix implementation for document node to remove out of root nodes.
* Support processing instruction node.
* Improve implementation for integer to omit d... - 06:21 AM Revision e044924e (git): rexml: Make more readable
- test/rexml/test_jaxen.rb: Use more meaningful name.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63220 b2dd03c8-39d4-4d8f-98ff-823fe69b080e - 06:21 AM Revision 1992cec5 (git): rexml: Fix a test bug
- test/rexml/test_jaxen.rb: Fix wrong assert_raise usage. Note that this code
isn't used yet.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63219 b2dd03c8-39d4-4d8f-98ff-823fe69b080e - 06:18 AM Revision daec80fe (git): rexml: Make more readable
- test/rexml/xpath/test_base.rb: Use here document for readability.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63218 b2dd03c8-39d4-4d8f-98ff-823fe69b080e - 04:02 AM Revision 7e9c19ad (git): test/ruby/test_io.rb: add closing recycled FD test
- Followup-to: r63216
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63217 b2dd03c8-39d4-4d8f-98ff-823fe69b080e - 03:12 AM Revision 645f7fbd (git): io.c: do not use rb_notify_fd_close close on recycled FD
- It is unsafe to release GVL and call rb_notify_fd_close after
close(2) on any given FD. FDs (file descriptor) may be recycled
in other threads immediately after close() to point to a different
file description. Note the distinction bet...
04/20/2018
- 10:53 PM Revision af72dcd9 (git): thread_sync: redo r62934 to use fork_gen
- Instead of maintaining linked-lists to store all
rb_queue/rb_szqueue/rb_condvar structs; store only a fork_gen
serial number to simplify management of these items.
This reduces initialization costs and avoids the up-front cost
of resett... -
09:38 PM Feature #14487 (Closed): [PATCH] simplify altstack and enable reuse with thread cache
- Applied in changeset trunk|r63213.
----------
simplify altstack and enable reuse with thread cache
Instead of allocating and registering the altstack in different
places, do it together to reduce code and improve readability.
When thre... - 09:38 PM Revision 730d257b (git): * 2018-04-21
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63214 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
- 09:38 PM Revision 475b4aa4 (git): simplify altstack and enable reuse with thread cache
- Instead of allocating and registering the altstack in different
places, do it together to reduce code and improve readability.
When thread cache is enabled, storing altstack in rb_thread_t
is wasteful and we may reuse altstack in the sam... -
06:34 PM Feature #6284: Add composition for procs
- yuroyoro (TOMOHITO Ozaki) wrote:
> Most languages do not define function composition in built-in operators, but provide them as function or method such as `compose`.
> ...
I do like the idea of using shovel as it's already fairly commo... -
03:23 PM Bug #14704 (Closed): Module#ancestors looks wrong when a module is both included and prepended in the same class.
- `Module#ancestors` looks wrong when a module is both included and prepended in the same class.
Here is the example script:
```ruby
module M3; end
module M1
include M3
end
module M2
prepend M3
end
class Sub
includ... -
01:42 PM Revision e72a86fc (git): _mjit_compile_send.erb: inline attr_reader call
- _mjit_compile_send_guard.erb: carve out the shared logic to invalidate
inlined method call
common.mk: update dependency for this change
test_jit.rb: add test for attr_reader optimization
* Benchmark
```
require 'benchmark_driver'
Be... -
01:40 PM Bug #14662: Stack consistency error in 2.5.1
- Unfortunately I haven't found a way to reproduce this without a large portion of our code running, and because the error happens in different places I haven't been able to track it down. I'll keep trying but I think it is unlikely that I...
-
01:23 PM Feature #14111: ArgumentErrorが発生した時メソッドのプロトタイプをメッセージに含む
- いくつか問題があるようです。
* `define_method` で定義されたメソッドで `ArgumentError` が起きるとSEGV
* `label` はメソッド名とは同じとは限らない
* 特異メソッドを持つオブジェクトをインスタンス変数にセットすると `Marshal.dump` できない
また、 `ArgumentError#to_s` は別ライブラリ(gem)で提供するということでしょうか。
「重箱の隅をつつく」ようですが、エラー発生か... -
11:52 AM Feature #14703 (Closed): [PATCH] net/imap: set SO_KEEPALIVE on TCP sockets
- Otherwise connections (commonly on IDLE, but it could be any
command) may never receive notifications of link errors.
Pretty trivial, will commit in a few days if no response.
-
10:11 AM Misc #14698: DevelopersMeeting20180517Japan
- shevegen (Robert A. Heiler) wrote:
> Just one comment:
> ...
This changed is just applied for the agenda page like https://bugs.ruby-lang.org/projects/ruby/wiki/DevelopersMeeting20180419Japan.
The wiki has an issue, other than commi... -
10:03 AM Bug #14702 (Closed): On Ruby 2.5.1, tracepoint isn't working on the file that is loaded by load_iseq
- On Ruby 2.5.1, when loading file with RubyVM::InstructionSequence.load_from_binary, TracePoint callback event is not working on loaded file.
## Steps to reproduce
Ruby version.
```
$ ruby -v
# => ruby 2.5.1p57 (2018-03-29 revisi... -
08:27 AM Feature #14701 (Rejected): If the object is not frozen, I want to be able to redefine the compound assignment operator.
- Use `append` instead of `+=` for arrays. Changing the behavior of `+=` would have too much compatibility problems from side-effect.
Matz.
-
07:10 AM Revision 1658fb3f (git): Update latest bundled gems:
- did_you_mean: 1.2.1
rake: 12.3.1
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63211 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
03:32 AM Bug #14634: Queue#push seems to crash after fork
- benoit@sqreen.io wrote:
> The customer just sent me another crasher report (segfault).
> It happens in another location in Ruby on
> `Module.const_defined?`. I looked at the ruby code while going
> through the crasher trace, and it ... - 03:22 AM Revision b456eab2 (git): variable.c: fix thread + fork errors in autoload
- This is fairly non-intrusive bugfix to prevent children
from trying to reach into thread stacks of the parent.
I will probably reuse this idea and redo r62934, too
(same bug).
* vm_core.h (typedef struct rb_vm_struct): add fork_gen coun... -
02:12 AM Feature #14352: Array#pack("M") Quoted-Printable with binary mode
- なるせさん
ご検討いただきありがとうございました。
mailライブラリとしてはこの結果をベースに修正案を出してみようと思います。 -
12:52 AM Bug #14700 (Rejected): Endless ranges don't seem to work properly with case statements
- Please use parentheses, like `when (0..)`. Unfortunately, it is difficult to allow this because it may cause a conflict with existing programs.
```
x = 1
case x
when 0..
3
p "#{ x } is in 0..3"
end
``` -
12:25 AM Revision 5b9bb500 (git): vm_core.h: adjust indent [ci skip]
- git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63209 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-
12:23 AM Revision db885d08 (git): range.c: step in bignum
- * range.c (range_step): honor step in bignum addition.
[Feature #12912]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63208 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
12:10 AM Revision ad5a6aa7 (git): range.c: fix fixnum loop condition
- * range.c (range_step): FIXABLE + FIXABLE never overflow, but may
not be FIXABLE. [Feature #12912]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63207 b2dd03c8-39d4-4d8f-98ff-823fe69b080e