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.

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

change pattern variable names in record to fieldname_v #38

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kenichi-asai
Copy link
Contributor

レコードのパターンに出て来るパターン変数の名前を フィールド名_v にしました。(これで issue #21 の問題のうち同じフィールド名が使われてしまう問題だけは回避できます。ただ、ほかのパターン変数と重なる名前に変更できてしまう問題は残っていますが。)

@kenichi-asai
Copy link
Contributor Author

コミット 2987815 で、型名の最後に _t をつけるようにしました。これで issue #21 は「同じパターン変数名にしようとしても拒否していない」点以外は解決したと思います。

あと、このコミットは generateUniqueName を変更したのですが、その中で明らかに不要と思われる2行を削除しました。大丈夫と思いますが、確認ください。

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