simple rescue+while+break should not use `throw` #4507

ko1 · 2021-05-18T08:14:38Z

609de71 fixes the issue by using
throw insn if ensure is used. However, that patch introduce
additional throw even if it is not needed. This patch solves
the issue.

This issue is pointed by @mame.

@mame

609de71 fixes the issue by using `throw` insn if `ensure` is used. However, that patch introduce additional `throw` even if it is not needed. This patch solves the issue. This issue is pointed by @mame.

simple rescue+while+break should not use throw

e473fa0

609de71 fixes the issue by using `throw` insn if `ensure` is used. However, that patch introduce additional `throw` even if it is not needed. This patch solves the issue. This issue is pointed by @mame.

ko1 force-pushed the rescue_break branch from 520371f to e473fa0 Compare May 21, 2021 04:28

ko1 merged commit 817764b into ruby:master May 21, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

simple rescue+while+break should not use `throw` #4507

simple rescue+while+break should not use `throw` #4507

ko1 commented May 18, 2021

simple rescue+while+break should not use throw #4507

simple rescue+while+break should not use throw #4507

Conversation

ko1 commented May 18, 2021

simple rescue+while+break should not use `throw` #4507

simple rescue+while+break should not use `throw` #4507