Bug #20296: Complex(:sym, exception: false) generate exception with weird timing

• Status: Closed
• Assignee: -
• Target version: -
• ruby -v: 3.3.0
• Backport: 3.0: REQUIRED, 3.1: REQUIRED, 3.2: DONE, 3.3: DONE

[MaxLap (Maxime Lapointe)]

Happens in Ruby 3.2.2, 3.2.3 and 3.3.0 (didn't test others).

Add this to a file and execute it as-is:

3.times { Complex(:sym, exception: false) }
puts "hi"

The output will be:

$ ruby local2.rb
hi
local2.rb:1:in `Complex': can't convert Symbol into Complex (TypeError)

3.times { Complex(:sym, exception: false) }
                  ^^^^^^^^^^^^^^^^^^^^^^
        from local2.rb:1:in `block in <main>'
        from local2.rb:1:in `times'
        from local2.rb:1:in `<main>'
local2.rb:1:in `Complex': can't convert Symbol into Complex (TypeError)

3.times { Complex(:sym, exception: false) }
                  ^^^^^^^^^^^^^^^^^^^^^^
        from local2.rb:1:in `block in <main>'
        from local2.rb:1:in `times'
        from local2.rb:1:in `<main>'
local2.rb:1:in `Complex': can't convert Symbol into Complex (TypeError)

3.times { Complex(:sym, exception: false) }
                  ^^^^^^^^^^^^^^^^^^^^^^
        from local2.rb:1:in `block in <main>'
        from local2.rb:1:in `times'
        from local2.rb:1:in `<main>'

So the exception is still reported (instead of being swallow), but the puts, which is after, had the time to execute first. I can even put a sleep() in between and the exceptions are still reported after the puts.

In a IRB, there is no exception at all and everything seems to work as expected.

Happens if I pass in an Array too. Doesn't seem to happen with bad strings.

There is also a quadratic behavior somewhere in there. Running the code in an IRB (which doesn't show exceptions), if you double the number of times in the loop, it takes 4x as long. This is probably related...

[MaxLap (Maxime Lapointe)]

• Description updated (diff)

[MaxLap (Maxime Lapointe)]

• ruby -v set to 3.3.0

[nobu (Nobuyoshi Nakada)]

• Backport changed from 3.0: UNKNOWN, 3.1: UNKNOWN, 3.2: UNKNOWN, 3.3: UNKNOWN to 3.0: REQUIRED, 3.1: REQUIRED, 3.2: REQUIRED, 3.3: REQUIRED

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Closed

Applied in changeset git|dc146babf47a84bbd1f176d766637d4a40327019.

---

[Bug #20296] Clear errinfo when exception: false

[k0kubun (Takashi Kokubun)]

• Backport changed from 3.0: REQUIRED, 3.1: REQUIRED, 3.2: REQUIRED, 3.3: REQUIRED to 3.0: REQUIRED, 3.1: REQUIRED, 3.2: REQUIRED, 3.3: DONE

ruby_3_3 8f1084db9b07cb74f99de70d6f8bb6076d27d8aa merged revision(s) dc146babf47a84bbd1f176d766637d4a40327019,f23d5028059078a346efc977287b669d494a5a3f,a0f7de814ae5c299d6ce99bed5fb308a05d50ba0.

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 3.0: REQUIRED, 3.1: REQUIRED, 3.2: REQUIRED, 3.3: DONE to 3.0: REQUIRED, 3.1: REQUIRED, 3.2: DONE, 3.3: DONE

ruby_3_2 90f4c5dc73af3fff76500dd4223792a6d9f58636 merged revision(s) dc146babf47a84bbd1f176d766637d4a40327019, f23d5028059078a346efc977287b669d494a5a3f, a0f7de814ae5c299d6ce99bed5fb308a05d50ba0.