Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Unused variable #894

Closed
satk0 opened this issue Feb 6, 2025 · 1 comment
Closed

Unused variable #894

satk0 opened this issue Feb 6, 2025 · 1 comment

Comments

@satk0
Copy link
Contributor

satk0 commented Feb 6, 2025

When compiling r2 with CFLAGS="-O0 -Werror -Wno-unused-result -Wno-stringop-truncation" @trufae has found this:

Image

I'll make a small PR fixing that.
satk0 added a commit to satk0/quickjs that referenced this issue Feb 6, 2025
@satk0
Remove unused variable, fix issue quickjs-ng#894
7424373
@satk0 satk0 mentioned this issue Feb 6, 2025
Merged
satk0 added a commit to satk0/quickjs that referenced this issue Feb 6, 2025
@satk0
Remove unused variable, fix issue quickjs-ng#894
f5f8f82
satk0 added a commit to satk0/quickjs that referenced this issue Feb 7, 2025
@satk0
Remove unused variable and add flags, fix issue quickjs-ng#894
b4acfd3
saghul pushed a commit that referenced this issue Feb 7, 2025
@satk0 @saghul
Remove unused variable and add flags, fix issue #894
02f06f6
@satk0
Copy link
Contributor Author

satk0 commented Feb 7, 2025

Thanks for merging my PR 💪 @saghul

@satk0 satk0 closed this as completed Feb 7, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@satk0