`else` block in `try` statement swallows exceptions
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript, python
- 領域
- compilers
調査の方向性
No source file or test is named. Start by tracing how the compiler translates the provided Python try/except/else example, then verify that an exception raised in the else block reaches the outer context instead of being swallowed; issue #99 may provide related history.
索引モデルが issue の本文から書いたものです。
説明
Exceptions raised in the else block should bubble out, but instead they currently get suppressed.
This:
try:
pass
except:
pass
else:
raise Exception('something wrong happened in the `else`')
compiles to:
try {
// pass;
try {
var __except0__ = Exception ('something wrong happened in the `else`');
__except0__.__cause__ = null;
throw __except0__;
}
catch (__except0__) {
}
}
catch (__except0__) {
// pass;
}
I suggest: set a temporary boolean variable to true before the try, then flip it to false as the very first statement inside the catch, and then have an if statement on that boolean as the very first statement after the catch block (it might be better to just always generate that if inside a finally in the JavaScript even if there is no finally in the Python, because it has to execute before any finally if it does exist).
(My reasoning for flipping the boolean as the first statement in the catch instead of the last statement in the try is that if for some reason ever a JavaScript implementation existed that raised an exception while trying to set that internal implementation book-keeping variable, I think it better that our generated code explode that up a level, instead of it being caught by the local except block, because logically that is an error within the try statement itself, not the block of code inside it.)
P.S. In similar situations in the future, would you prefer I reopened #99 instead of creating this new issue?
- 主要言語
- Python
- スター
- 2.9k
- フォーク
- 218
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
TranscryptOrg/Transcrypt のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
TranscryptOrg/Transcrypt#913 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
TranscryptOrg/Transcrypt#911 · コメント 2 件 ·
-
IS: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
TranscryptOrg/Transcrypt#908 ·
-
SUB: documentation
難易度 1/5 1時間未満 初心者へのやさしさ 62/100
TranscryptOrg/Transcrypt#656 · コメント 7 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 76/100
TranscryptOrg/Transcrypt#914 ·
TranscryptOrg/Transcrypt の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
zostera/django-bootstrap4#894 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
use-agent-os/agent-os#3276 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
zephyrproject-rtos/zephyr#119726 ·
-
area/auth bug comp/agent P3 platform/discord type/security
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
NousResearch/hermes-agent#117848 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
zilliztech/memsearch#759 ·