Skip to content

Remove unused variable, fix issue #894#895

Merged
saghul merged 1 commit intoquickjs-ng:masterfrom
satk0:remove-unused-var
Feb 7, 2025
Merged

Remove unused variable, fix issue #894#895
saghul merged 1 commit intoquickjs-ng:masterfrom
satk0:remove-unused-var

Conversation

@satk0
Copy link
Copy Markdown
Contributor

@satk0 satk0 commented Feb 6, 2025

resolves issue #894

@satk0 satk0 force-pushed the remove-unused-var branch from 7424373 to f5f8f82 Compare February 6, 2025 22:10
@satk0
Copy link
Copy Markdown
Contributor Author

satk0 commented Feb 6, 2025

I can add -Wno-unused-result -Wno-stringop-truncation to CFLAGS, wdyt?

@bnoordhuis
Copy link
Copy Markdown
Contributor

Good idea.

@satk0 satk0 force-pushed the remove-unused-var branch from f5f8f82 to b4acfd3 Compare February 7, 2025 13:52
@satk0
Copy link
Copy Markdown
Contributor Author

satk0 commented Feb 7, 2025

Done it, but idk if that's what you wanted 😅

@saghul saghul merged commit 02f06f6 into quickjs-ng:master Feb 7, 2025
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.

3 participants