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

Fix type-checking for the type argument of abi.decode #14319

Closed
ekpyron opened this issue Jun 13, 2023 · 3 comments · Fixed by #14347
Closed

Fix type-checking for the type argument of abi.decode #14319

ekpyron opened this issue Jun 13, 2023 · 3 comments · Fixed by #14347
Assignees
Labels
bug 🐛 selected for development It's on our short-term development
Milestone

Comments

@ekpyron
Copy link
Member

ekpyron commented Jun 13, 2023

Properly reject the few incorrectly allowed complex expressions with side-effects in the second argument of abi.decode.

@ekpyron ekpyron added this to the 0.8.21 milestone Jun 13, 2023
@ekpyron ekpyron added the selected for development It's on our short-term development label Jun 13, 2023
@matheusaaguiar matheusaaguiar self-assigned this Jun 13, 2023
@AnuragNishad12
Copy link

Hiii @ekpyron can u explain me whats the problem ?

@matheusaaguiar
Copy link
Collaborator

matheusaaguiar commented Jun 21, 2023

Hi @AnuragNishad12, thanks for your willingness to contribute to the project.
This issue is already being worked on (by myself :) ). Could search for another issue, please? I recommend searching for the ones involving documentation changes, if you don't have much experience with the compiler/C++ language. There are also many other issues which are about the compiler itself that you could choose if you feel comfortable with the task.
If you have any questions or need help figuring that out, please reach the team on the matrix channel.

@AnuragNishad12
Copy link

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 selected for development It's on our short-term development
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants