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 some warnings #4374

Closed
wants to merge 3 commits into from
Closed

Fix some warnings #4374

wants to merge 3 commits into from

Conversation

rootkea
Copy link
Contributor

@rootkea rootkea commented Oct 24, 2023

Hello!

This PR fixes some warnings like:

../app/src/util/vecdeque.h:193:24: warning: arithmetic on a pointer to void is a GNU extension [-Wpointer-arith]
    memcpy(newptr, ptr + (oldorigin * item_size), right_len * item_size);
                   ~~~ ^
../app/src/util/vecdeque.h:196:23: warning: arithmetic on a pointer to void is a GNU extension [-Wpointer-arith]
        memcpy(newptr + (right_len * item_size), ptr,
               ~~~~~~ ^
...
../app/src/server.c:303:45: warning: empty expression statement has no effect; remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
        ADD_PARAM("downsize_on_error=false");

rom1v pushed a commit that referenced this pull request Oct 24, 2023
To fix the warnings of stray `;`.

PR #4374 <#4374>

Signed-off-by: Romain Vimont <rom@rom1v.com>
rom1v added a commit that referenced this pull request Oct 24, 2023
PR #4374 <#4374>

Co-authored-by: Romain Vimont <rom@rom1v.com>
Signed-off-by: Romain Vimont <rom@rom1v.com>
rom1v pushed a commit that referenced this pull request Oct 24, 2023
PR #4374 <#4374>

Signed-off-by: Romain Vimont <rom@rom1v.com>
@rom1v
Copy link
Collaborator

rom1v commented Oct 24, 2023

Thank you 👍 Merged into dev:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants