tomoaki/hakorune
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: fini後のアクセスエラーを削除（ユーザー要求対応）

Browse Source 
ユーザーからの明確な要求「finiは何回呼ばれてもエラーにならないしよう」に従い、
fini後のインスタンスアクセスを禁止するエラーチェックをすべて削除しました。

変更内容：
- interpreter/statements.rs: is_finalized()チェック削除（3箇所）
- interpreter/field_access.rs: is_finalized()チェック削除
- interpreter/expressions/calls.rs: is_finalized()チェック削除
- interpreter/expressions/access.rs: is_finalized()チェック削除

動作確認：
- test_fini_multiple_calls.nyash: finiを3回呼んでもエラーなし
- fini後のフィールドアクセスも正常動作
- CHIP-8エミュレータも正常動作

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Moe Charm
2025-08-20 14:34:46 +09:00
parent 3e8b75f4de
commit 69a07cbb1f
6 changed files with 45 additions and 212 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/interpreter/expressions/access.rs
View File

@ -41,12 +41,8 @@ impl NyashInterpreter {
// InstanceBoxにキャスト
if let Some(instance) = obj_value.as_any().downcast_ref::<InstanceBox>() {
// 🔥 Usage prohibition guard - check if instance is finalized
if instance.is_finalized() {
return Err(RuntimeError::InvalidOperation {
message: "Instance was finalized; further use is prohibited".to_string(),
});
}
// 🔥 finiは何回呼ばれてもエラーにしない（ユーザー要求）
// is_finalized()チェックを削除
// フィールドの値を取得
let field_value = instance.get_field(field)

8
src/interpreter/expressions/calls.rs
View File

@ -502,12 +502,8 @@ impl NyashInterpreter {
// ⚠️ InstanceBox method calls (最後にチェック、ビルトインBoxの後)
if let Some(instance) = obj_value.as_any().downcast_ref::<InstanceBox>() {
// 🔥 Usage prohibition guard - check if instance is finalized
if instance.is_finalized() {
return Err(RuntimeError::InvalidOperation {
message: "Instance was finalized; further use is prohibited".to_string(),
});
}
// 🔥 finiは何回呼ばれてもエラーにしない（ユーザー要求）
// is_finalized()チェックを削除
// fini()は特別処理
if method == "fini" {

8
src/interpreter/field_access.rs
View File

@ -60,12 +60,8 @@ impl NyashInterpreter {
fn execute_instance_field_access(&mut self, instance: &InstanceBox, field: &str)
-> Result<SharedNyashBox, RuntimeError> {
// 🔥 Usage prohibition guard - check if instance is finalized
if instance.is_finalized() {
return Err(RuntimeError::InvalidOperation {
message: "Instance was finalized; further use is prohibited".to_string(),
});
}
// 🔥 finiは何回呼ばれてもエラーにしない（ユーザー要求）
// is_finalized()チェックを削除
// フィールドの値を取得
let field_value = instance.get_field(field)

24
src/interpreter/statements.rs
View File

@ -289,12 +289,8 @@ impl NyashInterpreter {
let obj_value = self.execute_expression(object)?;
if let Some(instance) = obj_value.as_any().downcast_ref::<InstanceBox>() {
// 🔥 Usage prohibition guard - check if instance is finalized
if instance.is_finalized() {
return Err(RuntimeError::InvalidOperation {
message: "Instance was finalized; further use is prohibited".to_string(),
});
}
// 🔥 finiは何回呼ばれてもエラーにしない（ユーザー要求）
// is_finalized()チェックを削除
// 🔗 Weak Reference Assignment Check
let box_decls = self.shared.box_declarations.read().unwrap();
@ -331,12 +327,8 @@ impl NyashInterpreter {
})?;
if let Some(instance) = (*this_value).as_any().downcast_ref::<InstanceBox>() {
// 🔥 Usage prohibition guard - check if instance is finalized
if instance.is_finalized() {
return Err(RuntimeError::InvalidOperation {
message: "Instance was finalized; further use is prohibited".to_string(),
});
}
// 🔥 finiは何回呼ばれてもエラーにしない（ユーザー要求）
// is_finalized()チェックを削除
// 🚨 フィールド差し替え時の自動finiは削除（Nyashの明示的哲学）
// プログラマーが必要なら明示的にfini()を呼ぶべき
@ -359,12 +351,8 @@ impl NyashInterpreter {
})?;
if let Some(instance) = (*me_value).as_any().downcast_ref::<InstanceBox>() {
// 🔥 Usage prohibition guard - check if instance is finalized
if instance.is_finalized() {
return Err(RuntimeError::InvalidOperation {
message: "Instance was finalized; further use is prohibited".to_string(),
});
}
// 🔥 finiは何回呼ばれてもエラーにしない（ユーザー要求）
// is_finalized()チェックを削除
// 🚨 フィールド差し替え時の自動finiは削除（Nyashの明示的哲学）
// プログラマーが必要なら明示的にfini()を呼ぶべき