Skip to content

Remove return from void functions#1439

Closed
andrjohns wants to merge 1 commit intoquickjs-ng:masterfrom
andrjohns:void-returns
Closed

Remove return from void functions#1439
andrjohns wants to merge 1 commit intoquickjs-ng:masterfrom
andrjohns:void-returns

Conversation

@andrjohns
Copy link
Copy Markdown
Contributor

Removes the return from a few void functions, as it was causing warnings under -Wpedantic:

  ./quickjs/quickjs.c:4573:5: warning: void function 'JS_FreeCString' should not return void expression [-Wpedantic]
  ./quickjs/quickjs.c:4578:5: warning: void function 'JS_FreeCStringRT' should not return void expression [-Wpedantic]
  ./quickjs/quickjs.c:4583:5: warning: void function 'JS_FreeCStringUTF16' should not return void expression [-Wpedantic]
  ./quickjs/quickjs.c:4588:5: warning: void function 'JS_FreeCStringRT_UTF16' should not return void expression [-Wpedantic]

@saghul
Copy link
Copy Markdown
Contributor

saghul commented Apr 3, 2026

See: #1404

@andrjohns
Copy link
Copy Markdown
Contributor Author

See: #1404

Ack sorry for not checking properly!

@andrjohns andrjohns closed this Apr 3, 2026
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.

2 participants