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
Fix segmentation fault when assigning to a parenthesized non-tuple expression of a tuple type #15084
Conversation
184d65f
to
ad74d10
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new condition is technically incorrect (or rather makes unnecessary assumptions), though cases that it would process incorrectly are currently impossible in the language, so the code is not buggy despite this. I think it would still be better to correct it before we merge the PR.
test/libsolidity/syntaxTests/tupleAssignments/empty_tuple_lhs_resolved.sol
Outdated
Show resolved
Hide resolved
0b0192c
to
0646e8f
Compare
0646e8f
to
c88b3cf
Compare
test/libsolidity/syntaxTests/tupleAssignments/empty_tuples_lhs_resolved.sol
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to tentatively approve it since my only remaining problem here is wording and comments, but would still be good if those could be corrected before we merge.
b8eb208
to
3ec1ed6
Compare
...ntaxTests/tupleAssignments/assignments_to_tuple_and_non_tuple_expressions_of_tuple_types.sol
Outdated
Show resolved
Hide resolved
...ntaxTests/tupleAssignments/assignments_to_tuple_and_non_tuple_expressions_of_tuple_types.sol
Outdated
Show resolved
Hide resolved
6efbf56
to
e53197d
Compare
e53197d
to
3b4d9bb
Compare
...ntaxTests/tupleAssignments/assignments_to_tuple_and_non_tuple_expressions_of_tuple_types.sol
Outdated
Show resolved
Hide resolved
a86dd89
to
2d0fcb0
Compare
…pression of a tuple type Co-authored-by: Kamil Śliwak <kamil.sliwak@codepoets.it>
2d0fcb0
to
5ffd39a
Compare
Fixes #15075