! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Fix compiler warnings (gcc 8)
Status:
RESOLVED: FIXED

Comments

Description Olivier Duchateau 2019-02-03 16:23:27 CET
Created attachment 8282 
Fix compiler warnings

With GCC 8 (no messages with Clang) I get these warnings:

> xfconf-channel.c: In function 'xfconf_channel_get_g_property':
> xfconf-channel.c:329:13: warning: this statement may fall through [-Wimplicit-fallthrough=]
>             g_value_set_boolean(value, channel->is_singleton);
>             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>xfconf-channel.c:331:9: note: here
>         default:
>         ^~~~

>xfconf-daemon.c: In function 'xfconf_daemon_backend_property_changed':
>xfconf-daemon.c:135:20: warning: assignment to 'XfconfDaemon *' {aka 'struct _XfconfDaemon *'} from incompatible pointer type 'GObject *' {aka 'struct _GObject *'} [-Wincompatible-pointer-types]
>     pdata->xfconfd = g_object_ref(G_OBJECT(user_data));
>                                ^
>xfconf-daemon.c:136:20: warning: assignment to 'XfconfBackend *' {aka 'struct _XfconfBackend *'} from incompatible pointer type 'GObject *' {aka 'struct _GObject *'} [-Wincompatible-pointer-types]
>     pdata->backend = g_object_ref(G_OBJECT(backend));
>                                 ^

Patch fixes this issue.
Comment 1 Git Bot editbugs 2019-02-13 01:58:37 CET
Olivier Duchateau referenced this bugreport in commit 0cb3bfa9adb383117251e0799beafcc7e89fcb10

Fix warnings reported by GCC 8 (Bug #15112)

https://git.xfce.org/xfce/xfconf/commit?id=0cb3bfa9adb383117251e0799beafcc7e89fcb10
Comment 2 Andre Miranda editbugs 2019-02-13 01:59:28 CET
LGTM, merged, thanks!

Bug #15112

Reported by:
Olivier Duchateau
Reported on: 2019-02-03
Last modified on: 2019-02-13

People

Assignee:
Brian J. Tarricone (not reading bugmail)
CC List:
2 users

Version

Version:
GIT Master

Attachments

Fix compiler warnings (1.50 KB, application/octet-stream)
2019-02-03 16:23 CET , Olivier Duchateau
no flags

Additional information