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

Парсинг Get\SetMemberNode #45

Closed
impworks opened this issue Feb 5, 2013 · 3 comments
Closed

Парсинг Get\SetMemberNode #45

impworks opened this issue Feb 5, 2013 · 3 comments
Assignees
Milestone

Comments

@impworks
Copy link
Owner

impworks commented Feb 5, 2013

Посмотрел в *MemberNode, а там только название member'а и присваиваемое значение в set-варианте. Когда разбирается выражение типа (a + b).mbr, куда записывается (a + b)?

@ghost ghost assigned ForNeVeR Feb 5, 2013
@ForNeVeR
Copy link
Collaborator

ForNeVeR commented Feb 5, 2013

Для GetMemberNode это точно должно работать - выражение должно быть записано в поле Expression, это проверяется тестами. Может быть, есть баг для SetMemberNode? Я попробую проверить.

@ForNeVeR
Copy link
Collaborator

ForNeVeR commented Feb 6, 2013

Проверил, и даже добавил тест для SetMemberNode - вроде бы всё ок.

Конечно, тесты могли бы заглючить, если бы у нас были неправильно написаны методы Equals для узлов, но я посмотрел реализацию сравнения для MemberNodeBase - с ней, вроде бы, тоже всё в порядке.

@impworks, если что-то работает не так, как ожидается - дай знать.

@impworks
Copy link
Owner Author

impworks commented Feb 6, 2013

Все верно - я забыл про это поле. Тикет закрываю.

@impworks impworks closed this as completed Feb 6, 2013
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

No branches or pull requests

2 participants