Activity
From 10/03/2023 to 10/09/2023
10/09/2023
-
10:53 PM Revision 7de6908a (git): LLDB: Print actual node structure [ci skip]
-
10:48 PM Feature #19915: URI::HTTP.build accepts user: and password: keyboard arguments, but do not populate #user or #password
- I would expect either for unused options to raise an ArgumentError, or for `user:` and `password:` to be accepted and used. Since `URI::HTTP.build` is actually accepting a Hash [1], this makes it harder to validate the given Hash keys an...
-
12:25 AM Feature #19915: URI::HTTP.build accepts user: and password: keyboard arguments, but do not populate #user or #password
- `URI::HTTP.build` does not accept keyword arguments, it accepts an array/hash. If you pass keyword arguments, they get turned into a hash, similarly to any Ruby method that does not explicitly accept keyword arguments. The method only r...
-
08:15 PM Revision c3fc70cc (git): [rubygems/rubygems] Exempt default gem 'etc' in setup tests for rubies before 3.2 on windows
- https://github.com/rubygems/rubygems/commit/834ae8a29a
-
03:09 PM Revision a8d71535 (git): Pre-allocate `NODE_DEFN`/`NODE_DEFS` nodes
- To fix the order of node IDs, for typeprof tests.
-
- Use `NODE_DEF_TEMP` as the wrapper of `NODE_DEFS` in ripper, not only
of `NODE_DEFN`, so that `endless_method_name` works consistently. -
- 03:05 PM Revision 6c952ac6 (git): testing_ruby.md: Document running test suites on s390x CPU architecture [ci skip] (#8589)
-
03:03 PM Misc #19883: DevMeeting-2023-10-12
- * [Feature #13933] Add Range#empty?
* "empty?" is a basic concept for core classes
* Related to the implementation of `Range#overlap?` (empty_region_p) so why not make it a public interface?
* For beginless ranges we cannot use ... - 11:54 AM Revision f44cee90 (git): [rubygems/rubygems] Avoid duplicates -rbundler/setup in RUBYOPT with Ruby preview
- When using a Ruby preview the require path of `bundler/setup` is
similar to `-r/opt/ruby3.3.0-preview2/lib/ruby/3.3.0+0/bundler/setup`.
The special character `+` in the string makes the Regexp fail,
leading to multiple addition of the sa... -
10:16 AM Revision 0fdee133 (git): [ruby/prism] Attach the ast node to the scope
- So when building instruction sequences for a scope we can reference
items from the ast node that requires the scope. This is useful for for
loops, where the local variable tables from the parent scope will need
to be referenced.
https:/... -
-
08:13 AM Bug #19916 (Closed): URI#to_s can serialize to a value that doesn't deserialize to the original
- It appears that when we're serializing a URI to string, we don't check/confirm that it is represented in a form that can be deserialized back into the original. I think it's fair to expect that serialize+deserialize produces an object th...
-
-
-
- In parse.y, node can be NODE_DEF_TEMP or NODE_EXITS.
Hit "unknown node" without taking care of them.
For example `ruby --dump=y lib/cgi/util.rb` causes
"[BUG] unknown node: 107". - 03:23 AM Revision a78acf18 (git): Bump ruby/setup-ruby from 1.154.0 to 1.155.0
- Bumps [ruby/setup-ruby](https://github.com/ruby/setup-ruby) from 1.154.0 to 1.155.0.
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Commits](https://github.com/ruby/setup-ruby/compare/52b8784594ec115fd17094752708121dc5... - 03:16 AM Revision a149ca51 (git): Bump ossf/scorecard-action from 2.2.0 to 2.3.0
- Bumps [ossf/scorecard-action](https://github.com/ossf/scorecard-action) from 2.2.0 to 2.3.0.
- [Release notes](https://github.com/ossf/scorecard-action/releases)
- [Changelog](https://github.com/ossf/scorecard-action/blob/main/RELEASE.md...
10/08/2023
-
- I noticed that `URI::HTTP.build` accepts the `user:` and `password:` keyword arguments, but does not actually set the `user` or `password` attributes of the built URI object. It does however correctly accept a `userinfo:` keyword argumen...
-
07:31 PM Feature #19884: Make Safe Navigation Operator work on classes - ______________________________________________
-
10:40 AM Feature #19884: Make Safe Navigation Operator work on classes
- nobu (Nobuyoshi Nakada) wrote in #note-10:
> Rather the extension of `defined?` operator which returns the value (instead of the expression type string) or nil.
> ...
Yes, that would be nice. -
- What you want doesn't seem the extension of the safe navigation operator.
p8 (Petrik de Heus) wrote:
> If a constant isn't defined it will raise a NameError:
> ...
Rather the extension of `defined?` operator which returns the value (ins... -
04:30 AM Feature #19884: Make Safe Navigation Operator work on classes
- sawa (Tsuyoshi Sawada) wrote:
> > > > You stated that you want &. to work on classes, but that does not make sense. If ActiveRecord::Base is not defined, it is not a class. How would Ruby know that it is (supposed to be) a class?
p8 ... -
-
- pbvdon (pbv pbv) wrote in #note-3:
> Any solution for this ?
You can try 2.5.9 which had been released as the final 2.5 release, or more recent versions.
Anyway we can do nothing unless we know how to reproduce it at least, even if ... -
- I've forgot about inline option and comment, so it was complicated than I thought first.
https://github.com/nobu/ruby/tree/warn-multiline-regexp -
04:17 AM Revision 2b6228be (git): [rubygems/rubygems] Reduce allocations when parsing compact index
- This still allocates a ton (a string for each line, plus a bunch of
splits into arrays), but it helps a bit when Bundler has to go through
dependency resolution.
```
==> memprof.after.txt <==
Total allocated: 194.14 MB (2317172 objects)... -
- It has been documented.
https://github.com/ruby/rdoc/blob/master/lib/rdoc/parser/ruby.rb#L141-L142
> Note that by default, the :method: directive will be ignored if there is a
> ...
-
- It appears that the next method takes precedence over the `:method:` directive.
And that directive seems like parsed as a simple document. -
- With a simplified example like bellow, `private` does not affect the results.
```ruby
class Bug19914
##
# :method: foo
#
# :call-seq: foo(*args)
#
# Foo
private
end
```
-
01:36 AM Bug #19914: RDoc :method: directives swallowed by immediate private keyword
- zzak (zzak _) wrote:
> @jonathanhefner Recently discovered this bug in RDoc and fixed it in Rails, so I'm reporting it here on their behalf.
> ...
This should probably be reported to https://github.com/ruby/rdoc/issues instead. -
12:15 AM Bug #19914 (Feedback): RDoc :method: directives swallowed by immediate private keyword
- @jonathanhefner Recently discovered this bug in RDoc and fixed it in Rails, so I'm reporting it here on their behalf.
Examples:
* https://github.com/rails/rails/commit/c8b3642eded5a25ee5cc11c48395bed05f92424e
* https://github.com/...
10/07/2023
-
- From running in a random rails app I have locally, here are the changes
1) for `bundle lock --update --bundler` (forcing Bundler to go through
dependency resolution)
```
==> memprof.after.txt <==
Total allocated: 2.98 MB (48307 objects... -
- sawa (Tsuyoshi Sawada) wrote in #note-6:
> Then you should edit and write that in the description.
I've updated the description. Hopefully it's more clear now.
> ...
```ruby
defined?(UndefinedConstant) # returns nil instead of... -
- p8 (Petrik de Heus) wrote in #note-5:
> > Furthermore, the specification you are asking for is not clear. What Exception classes do you want `&.` to rescue?
> ...
Then you should edit and write that in the description.
> `defined?` ... -
- sawa (Tsuyoshi Sawada) wrote in #note-1:
> Currently, you can do `ActiveRecord::Base.some_method rescue nil`
That would also rescue UndefinedMethodError, ArgumentError, and any error occuring in `some_method`.
> ...
`NameError`
... -
- There's multiple examples in the Rails code base:
https://github.com/rails/rails/blob/7f7f9df8641e35a076fe26bd097f6a1b22cb4e2d/railties/lib/rails/generators/named_base.rb#L139
https://github.com/rails/rails/blob/7f7f9df8641e35a076fe26bd... -
02:24 AM Feature #19884: Make Safe Navigation Operator work on classes
- @rubyFeedback
It's common in library code, that want's to be flexible to different configurations of the target application. E.g. https://github.com/rubocop/rubocop/blob/a455e9d55771f1e3dfea0cc4183e66f9632b431c/lib/rubocop/lockfile.rb... -
-
-
-
-
-
-
03:25 AM Misc #19883: DevMeeting-2023-10-12
- * [Misc #18984] Doc for Range#size for Float/Rational does not make sense (kyanagi)
* Range#size may return a non-nil value even if the range can't be iterated.
* Should it return nil?
* [Feature #18515] Add Range#reverse_each imp... -
- It seems that there is a similar issue with beginless ranges.
```
(..0).size # => Infinity
(..0).count # => Infinity
(..0).each {} # => can't iterate from NilClass (TypeError)
```
Is it reasonable to say the size is infinite ev...
10/06/2023
-
10:20 PM Feature #19905: Introduce `Queue#peek`
- `peek` is inherently racy in that the value observed might be already popped by the time the value is returned.
If it's for observability and debugging I think `to_a` or `each` makes more sense, it's for sure useful to see other values t... -
05:28 PM Feature #19905: Introduce `Queue#peek`
- I'm just curious if it might be better for #peek to lock to avoid showing something that has already popped or other parallel access issues? Is there significant advantage having it not lock?
-
04:04 PM Bug #19913: Getting interpreter error
- pbvdon (pbv pbv) wrote in #note-2:
> nobu (Nobuyoshi Nakada) wrote in #note-1:
> ...
Hi nobu,
Any solution for this ?
-
- nobu (Nobuyoshi Nakada) wrote in #note-1:
> Crashed ruby seems 2.5, which has been ended.
> ...
nobu (Nobuyoshi Nakada) wrote in #note-1:
> Crashed ruby seems 2.5, which has been ended.
> ...
Yes its the below :-
[BUG] Segmentatio... -
- Crashed ruby seems 2.5, which has been ended.
The version should be reported before the "Control frame information" line. -
- ruby --version
ruby 2.7.0p0 (2019-12-25 revision 647ee6f091) [x86_64-linux-gnu]
gem --version
3.1.2
os - ubuntu20
```
-- Control frame information -----------------------------------------------
c:0033 p:---- s:0166 e:00016... -
- From https://bugs.ruby-lang.org/issues/19883#change-104825
> Seems odd for a standard library to attempt to load a non-standard library first, and fallback to loading a standard library.
Agreed.
I think either lib/readline.rb shou... -
- `Array.pack1(obj, format) -> String` sounds weird since there is nothing about Array in there.
I think `String.pack1(format, obj)` is the best option.
(`String#pack1(obj)` is confusing because of existing `Array#pack(format)` which has... - 02:10 PM Revision 7eccc13c (git): [ruby/prism] Emit error when assigning to a numbered parameter
- https://github.com/ruby/prism/commit/66248ac2f6
-
- * 'j' and 'J' are not pointer directives.
* size of 'j' and 'J' are platform-dependent, may not be 64-bit.
* mention corresponding C types when native-size modifier is added.
Co-Authored-By: BurdetteLamar <burdettelamar@yahoo.com> -
- A transparent scope is a scope that cannot have local variables added to
it's local table. When a local is added to it's table, it instead gets
added to the first non-transparent parent scope.
This is used in for loops to ensure the cor... -
- Note that complex regexps use often multiple lines, with `x` option.
So, it might be a possible idea to warn if the `x` option is not specified for a multi-line regexp. -
05:59 AM Misc #19912: [Small potential for slight improvement in regards to trailing / that were omitted in a regex]
- Forgot to clarify what I meant with:
> I assume ruby has to decide what is going on, but usually
> ...
Here I meant that the ruby parser may have to identify how
/ are used, e. g. for divisions. In the context above, though,
ruby p... -
- Not sure if I should file this under bug or not; it is probably not a bug per se
and just the current behaviour. So I filed it under Misc here.
So let me copy/paste the code next:
alias e puts
i = 'abc'
case ... -
- Where `assert_match` converts string matcher argument to regexp first
with escaping, `assert_include` does the same thing simpler.
https://github.com/ruby/openssl/commit/81007e0a49 -
- https://github.com/ruby/openssl/commit/9a6e24daaf
-
-
- `RbConfig::SIZEOF` has been added later than the time this check was
committed first. -
- It is required in the always defined class level.
-
-
05:38 AM Revision 6a0c13c1 (git): [rubygems/rubygems] Update man page for bundle exec to reflect default flag value
- `--keep-file-descriptors` is true by default.
https://github.com/rubygems/rubygems/commit/b28e88e228 - 01:57 AM Revision 69b024d7 (git): [ruby/prism] Add full_name to ConstantPathNode and ConstantPathTargetNode
- https://github.com/ruby/prism/commit/b390553028
10/05/2023
-
-
-
10:28 PM Bug #19911 (Feedback): require slower with ruby/3.2.2
- You're measuring a lot of other things by using the `time` command to benchmark the script. It's more accurate to measure the time inside a script.
```
$ ruby -v -e 't = Time.new; require "json"; puts "%.3fs" % (Time.new - t)'
ruby 2.7.... -
09:14 PM Bug #19911: require slower with ruby/3.2.2
- On further debugging we found that the following is speeding up the require and file read operation
time ruby --disable-gems -e 'require "json"' ... -
- Also i have seen this happeneing in other versions/systems one other thing i tested was with ubuntu.
-
- we tested require with ruby 3.2.2 and ruby 2.7.2
and its seems to slowdown considerabily
the test case:
Ruby 3.2.2
command: time ruby -e 'require "json"'
time: user=0.19s system=0.02s cpu=99% total=0.204
Ruby 2.7.2
command: ti... -
09:31 PM Revision 8cb906d7 (git): YJIT: Simplify linker flavor check
- By not allowlisting every OS that could have a GNU flavor linker,
it's also a bigger tent -
- * [Feature #18573] Object#pack1 (jeremyevans0)
* For the same reason as String#unpack1, saves an array allocation when packing a single object.
* I think it would be useful to have this feature, but I prefer Array.pack1 to Object#p... -
- I developed a patch for this that implemented the feature using `Array.pack1` and was going to create a new feature request for it, but I'm glad to see there already is an existing feature request for it. Here's my pull request for it: ...
-
09:17 PM Revision 13bde94a (git): YJIT: shink local types from 8 to 4 bytes (#8597)
- * Shink local types from 8 to 4 bytes, context from 21 to 17 bytes
Use repr(packed)
* Add comment about Type being limited to 4 bits -
-
-
- Previously, at the end of `leave` we did
`*caller_cfp->sp = return_value`, like the interpreter.
With future changes that leaves the SP field uninitialized for C frames,
this will become problematic. For cases like returning from
`rb_fun... -
-
- This node type was added for the multi-value experiment back in 2004.
The feature itself was removed after a few years, but this is its
remnant. -
- ```
yjit/yjit.mk:48: Extraneous text after `ifneq' directive
``` -
05:23 PM Revision fd22d5de (git): YJIT: Build on BSD platforms with GNU make (#8591)
- 04:35 PM Revision 8d8fe543 (git): [ruby/prism] Check for duplicate parameter names in destructured params
- https://github.com/ruby/prism/commit/c3438aabea
-
- Although the problem seems avoided (or postponed maybe) now, this would still make sense, I think.
Since the ruby 2.6 bundled with macOS is configured to `--enable-shared` too, there is no reason for us to disable the `shared` option.
... - 03:12 PM Revision 38cb92a0 (git): [ruby/prism] Check for duplicate names in shadow args
- https://github.com/ruby/prism/commit/e100c6b8ae
-
01:19 PM Feature #19430: Contribution wanted: DNS lookup by c-ares library
- I've prototyped a patch that creates a temporal pthread whenever `Addrinfo.getaddrinfo` is called, and runs `getaddrinfo(3)` in the pthread.
https://gist.github.com/mame/d7b4d8eaa3b0c3016838c33eea752cbb
(This patch is incomplete; t... - 11:25 AM Revision fb9795cf (git): [ruby/prism] Fix parsing shadow args in lambda blocks
- https://github.com/ruby/prism/commit/53efaf252d
-
- Each bit run is upto the right shift count, so the each mask does not
need more upper bits. - 07:44 AM Revision f087f2c7 (git): [ruby/stringio] StringIO#pread: handle 0 length like IO#pread
- (https://github.com/ruby/stringio/pull/67)
Fix: https://github.com/ruby/stringio/issues/66
If length is 0, IO#pread don't even try to read the IO, it simply return
the buffer untouched if there is one or a new empty buffer otherwise.
... - 07:11 AM Revision 9d58f938 (git): [ruby/net-http] Net::HTTPResponse nil checking
- Fix nil handling in read_body and stream_check.
Fixes: #70
https://github.com/ruby/net-http/commit/36f916ac18 - 07:00 AM Revision d088b9f7 (git): Update bundled gems list at 2023-10-05
-
-
-
-
-
-
-
-
10/04/2023
-
-
- ElseNode looks to have been implemented at the same time as IfNode, but
was resulting in a stack underflow error.
The following is from the test code
```
if foo
bar
end
```
```
❯ make run
compiling compile.c
linking miniruby
./minir... -
-
-
-
-
-
-
05:00 PM Revision 488c0ed0 (git): [ruby/prism] Remove trailing parenthesis in Location#pretty_print
- https://github.com/ruby/prism/commit/8eaa199a28
-
03:49 PM Bug #19909: s390x zlib different deflate algorithm producing a different compressed byte stream causing test failures
- I sent PR to ruby/ruby to document this issue.
https://github.com/ruby/ruby/pull/8589
-
-
02:22 PM Revision e0c66b47 (git): [ruby/prism] [rust] write flag accessor functions
- https://github.com/ruby/prism/commit/f2333ba4c8
-
- https://github.com/ruby/prism/commit/630af5cb0e
-
02:02 PM Revision cd0b8d24 (git): [DOC] Fix typo in docs of IO: `#.` -> `$.`
-
12:13 PM Revision b43cc51d (git): [ruby/irb] Clear all context usages in RubyLex
- (https://github.com/ruby/irb/pull/684)
After this change, `RubyLex` will not interact with `Context` directly
in any way. This decoupling has a few benefits:
- It makes `RubyLex` easier to test as it no longer has a dependency on
`Co... -
-
08:22 AM Bug #19910: Set#delete_if behavior inconsistent with Array/Hash
- I was just looking at this, and it seems like the behaviour used to be as you expected but changed in v2.1, https://github.com/ruby/ruby/commit/2849ee5d18360f26b55643a2643a1c597376865d
-
06:46 AM Bug #19893: OpenStruct#respond_to? when true fails with NameError
- Thank you! That's it.
-
- Co-authored-by: Alan Wu <XrXr@users.noreply.github.com>
-
05:12 AM Revision 6d28f969 (git): [DOC] Fix description for `rb_postponed_job_register_one()`
- The current documentation for `rb_postponed_job_register_one()` is
explaining the differences with itself, where it should be explaining
the differences with `rb_postponed_job_register()`. -
- 02:51 AM Revision 2e1762c9 (git): Update default gems list at e501613efa2b010d6d697a4df264a2 [ci skip]
-
- https://github.com/ruby/bigdecimal/commit/c47802e813
-
- The test for integer part was separated at dc54574adefe.
10/03/2023
-
- * YJIT: Add counter to measure how often we compile "cold" ISEQs (#535)
Fix counter name in DEFAULT_COUNTERS
YJIT: add --yjit-cold-threshold, don't compile cold ISEQs
YJIT: increase default cold threshold to 200_000
Remove rb_yjit_ca... -
06:46 PM Revision d47af931 (git): [ruby/prism] Provide pm_parser_err_current and pm_parser_err_previous
- https://github.com/ruby/prism/commit/4828c96939
-
- https://github.com/ruby/prism/commit/4f107d8962
-
05:16 PM Revision accda74c (git): [rubygems/rubygems] Prevent gem activation in standalone mode
- As discussed in https://github.com/rubygems/rubygems/issues/6273#issuecomment-1449176658
The `gem` method behaves awkwardly in standalone mode. Assuming bundler
isn't loaded at all, a call to gem might activate a gem that is not part
of... -
- YJIT: Remove spill_temps from jit_prepare_routine_call
-
- Co-authored-by: Nobuyoshi Nakada <nobu@ruby-lang.org>
- 03:12 PM Revision 32baa2c7 (git): Update default gems list at 5dfbf1a3814c4d84a5ae375751f5f9 [ci skip]
-
03:10 PM Revision 5dfbf1a3 (git): [ruby/reline] Bump version to 0.3.9
- (https://github.com/ruby/reline/pull/594)
https://github.com/ruby/reline/commit/b6fb72718a -
-
09:42 AM Revision ed0661e6 (git): [rubygems/rubygems] Don't re-resolve with prereleases if unlocked gem has no prereleases
- https://github.com/rubygems/rubygems/commit/d76dc70d90
-
- https://github.com/rubygems/rubygems/commit/64e7a2656a
-
07:31 AM Revision b4ab013b (git): [DOC] Link fixes
-
07:13 AM Revision 25b536cc (git): [rubygems/rubygems] Fix invalid links in documentation. - wrap ENV variables in <code> - fix rubygems.org link - fix zenspider.com link
- https://github.com/rubygems/rubygems/commit/9eaac94a63
-
07:13 AM Revision 9ffd659d (git): [rubygems/rubygems] 🐛 Specification of branch or ref with tag is ambiguous
- - Specs for GitProxy were incorrect and insufficient
- Specs are now correct and less insufficient
https://github.com/rubygems/rubygems/commit/63d0a8cfd0 -
- https://github.com/rubygems/rubygems/commit/2851e051c3
-
- https://github.com/rubygems/rubygems/commit/1e487e1337
-
- (https://github.com/ruby/csv/pull/283)
https://github.com/ruby/csv/commit/af64a15b2f - 07:13 AM Revision 2325e1cd (git): [ruby/csv] Add CSV::InvalidEncodingError
- (https://github.com/ruby/csv/pull/287)
To handle encoding errors in CSV parsing with the appropriate error
class
https://github.com/ruby/csv/commit/68b44887e5 -
- Remove the bad example that can lead to misunderstanding as if this
precision is defined in Ruby. -
-
-
12:08 AM Revision 3b47fb2c (git): [rubygems/rubygems] Update suggested variable for bindir
- https://github.com/rubygems/rubygems/commit/f9cc6fed25