Skip to content

query: @db が nil のまま eval していたのを直す - #340

Merged
znz merged 2 commits into
rurema:masterfrom
znz:query-command-db
Sep 26, 2026
Merged

znz merged 2 commits into
rurema:masterfrom
znz:query-command-db

Conversation

@znz

@znz znz commented Sep 12, 2026

Copy link
Copy Markdown
Member

Refs #335(不具合 5)

概要

bitclust query は QueryCommand#exec が Subcommand#exec(DB を開く)を呼んでいなかったため @db が nil のまま eval され、'@db.classes.size' でも undefined method for nil で使えませんでした。

変更点

  • super を呼んで @db を用意する
  • doc/usage.md に query の説明(スクリプトからは @db で参照)と例を追加

検証

  • 新規 test/test_query_command.rb(2 件: MethodDatabase / --capi で FunctionDatabase)・全テスト green
  • 実 DB(3.4)で query '@db.classes.size' → 1332、'@db.fetch_class("Array").superclass.name' → "Object"

🤖 Generated with Claude Code

znz and others added 2 commits September 12, 2026 11:54
QueryCommand#exec が Subcommand#exec(DB を開く)を呼んでいなかったため、
'@db.classes.size' でも undefined method for nil になり使えなかった。
super を呼び、usage.md にスクリプトの書き方(@db で参照)と例を書く。

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
経緯はコミットログと PR に残す。

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@znz

znz commented Sep 25, 2026

Copy link
Copy Markdown
Member Author

#339 の指摘「経緯はコード中に残さない」をこの PR にも適用し、f500f10 でテストファイル冒頭の経緯説明を削除しました。

🤖 Generated with Claude Code

@znz
znz merged commit f6368e0 into rurema:master Sep 26, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant