Skip to content

Cleanup#2089

Merged
jubalh merged 17 commits intomasterfrom
cleanup
Feb 23, 2026
Merged

Cleanup#2089
jubalh merged 17 commits intomasterfrom
cleanup

Conversation

@jubalh
Copy link
Member

@jubalh jubalh commented Feb 20, 2026

Cleanup regarding compiler warnings.

Fix `type qualifiers ignored on function return type`.
The gpgme_set_passphrase_cb function expects a function
pointer of type gpgme_passphrase_cb_t.

The gpgme_passphrase_cb_t typedef specifies a function that returns
gpgme_error_t so return GPG_ERR_NO_ERROR instead of 0.
GDestroyNotify takes a void* and returns void.

Wrap xmpp_stanza_release to get rid of the incompatible function type
warning.
`name` of the struct `color_def` was not initialized but is also not
needed in this function.
The `arg_description` field was forgotten in 034a985.
Let's initialize them with NULL to make the compiler happy.
g_strstr_len() actually returns a gchar* to the position. So it is not
like in the case of strlen() where it returns the number of bytes.
Fixing a couple of -Wsign-compare warnings.
@jubalh jubalh added this to the next milestone Feb 20, 2026
@jubalh jubalh self-assigned this Feb 20, 2026
@jubalh jubalh added the cleanup label Feb 20, 2026
Explicitly cast the void pointer returned by g_hash_table_lookup
to the expected function pointer signature.

Make `-Wpedantic` happy.
Seems like in signal address.device_id is int32 for some reason.
@jubalh jubalh modified the milestones: next, next-next Feb 20, 2026
@jubalh jubalh merged commit 657de5f into master Feb 23, 2026
7 of 9 checks passed
@jubalh jubalh deleted the cleanup branch February 23, 2026 10:11
@jubalh jubalh modified the milestones: next-next, next Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant