Skip to content

Conversation

@arakur
Copy link
Contributor

@arakur arakur commented Oct 24, 2023

What

括弧を含むプロパティ/インデクスアクセスおよび関数呼び出しの構文解析のバグを修正しました.
(a.b).c のように括弧の内側および外側にプロパティ/インデクス/コールチェインが存在する場合,内側のコールチェインが無視されて a.c のような式と同様にパースされてしまっていたため,これを修正し a.b.c と等価になるようにしました.

Additional info

プロパティ/インデクス/コールチェインそれぞれの場合と,2 以上の長さのチェインを含む場合についてテストケースを作成しました.

Issue

#440

marihachi added a commit that referenced this pull request Oct 24, 2023
@FineArchs
Copy link
Member

このまま何も無ければ3日後に私の判断でマージしますが、オーナー @syuilo さんの判断を待ちたいという場合はご一報下さい。

また、お時間あればCHANGELOGへの追記をお願いします。

Copy link
Contributor

@marihachi marihachi left a comment

Choose a reason for hiding this comment

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

この修正で良いと思います

@FineArchs FineArchs merged commit fe952d9 into aiscript-dev:master Oct 28, 2023
@FineArchs
Copy link
Member

ありがとうございます!🙏

FineArchs added a commit to FineArchs-org/aiscript that referenced this pull request Nov 25, 2023
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.

3 participants