Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

インタプリタのエラーハンドリングhookでabortしないようにしたい #710

Closed
kakkokari-gtyih opened this issue Jul 5, 2024 · 3 comments · Fixed by #711
Closed

インタプリタのエラーハンドリングhookでabortしないようにしたい #710

kakkokari-gtyih opened this issue Jul 5, 2024 · 3 comments · Fixed by #711

Comments

@kakkokari-gtyih
Copy link

kakkokari-gtyih commented Jul 5, 2024 •
edited
Loading

AiScriptのコードがエラーで停止しても同じインタプリタを使いまわす処理があるが、err hookが設定されている場合に限ってabortが呼ばれてしまい、以降に別のAiScriptコードを実行させる際にも全く動作しなくなる問題を回避したい

Related to misskey-dev/misskey#14117, misskey-dev/misskey#14050
@kakkokari-gtyih kakkokari-gtyih mentioned this issue Jul 5, 2024
Closed
5 tasks
@FineArchs
Copy link
Member

「一箇所がエラー出てるのに他の所が動きっぱなしだとなんかややこしいことになりそう？」という感じでabortをつけた覚えがありますが、まあやり過ぎでしたね。

オプトインでabortする設定にできるようにするくらいはしたいかもです。

@FineArchs FineArchs mentioned this issue Jul 5, 2024
Merged
@kakkokari-gtyih
Copy link
Author

kakkokari-gtyih commented Jul 5, 2024 •
edited
Loading

オプトインでabort

この挙動に頼っている実装（そんなものあるのか？）が壊れる可能性があるので逆でもいい気がした

@kakkokari-gtyih kakkokari-gtyih linked a pull request Jul 5, 2024 that will close this issue
デフォルトではエラーハンドラ呼び出し時abortを行わないように #711
Merged
@FineArchs
Copy link
Member

オプトインでabort

この挙動に頼っている実装（そんなものあるのか？）が壊れる可能性があるので逆でもいい気がした

仕様のデザインとしては、より一般的であろうユースケースの方をデフォルトにしたい気持ちはあるんですよね
とはいえ破壊的は破壊的なので、一旦オプトアウトの形にしてnextからオプトインに入れ替えるとかが良さそうですかね？

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@kakkokari-gtyih @FineArchs