Skip to content

Conversation

@cacapon
Copy link
Owner

@cacapon cacapon commented May 5, 2025

fixed #213

概要

tokenizerが元々"a""b"["a","b"]別トークンで扱っていたのを["\"a\"\b\""]という形で一つのトークンにまとめる形にしました。

クオートをなくす処理についてはParserの責務ですので、このプルリクエストでは対応していません。
詳しくはー>#200

確認方法

make./minishで確認で動作可能ですが、効果を確認するのは難しいです。
一応対応前後でechoの挙動が変わります。

  • 修正前: echo "a""b" -> "a" "b" ※token区切りでスペースを入れて出力しているため、修正前はスペースが入る
  • 修正後: echo "a""b" -> "a""b"

Copy link
Collaborator

@dayano74 dayano74 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

動作に問題はないと思います。

@dayano74 dayano74 merged commit 06b9eac into main May 5, 2025
2 checks passed
@dayano74 dayano74 deleted the feature/213-concat-quote-token branch May 5, 2025 08:58
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.

tokenizerでつながるクオートを一つのTokenとして扱う

3 participants