After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 711835 - Clang build in broken: void function 'cal_comp_transfer_item_to' should not return a value
Clang build in broken: void function 'cal_comp_transfer_item_to' should not r...
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: general
3.11.x (obsolete)
Other Linux
: Normal major
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2013-11-11 12:11 UTC by Vadim Rutkovsky
Modified: 2013-11-11 12:37 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Bug #711835 - Clang build in broken: void function 'cal_comp_transfer_item_to' should not return a value (1.10 KB, patch)
2013-11-11 12:19 UTC, Fabiano Fidêncio
committed Details | Review

Description Vadim Rutkovsky 2013-11-11 12:11:24 UTC
Commit https://git.gnome.org/browse/evolution/commit/calendar/gui?id=ed2bc85f4fe13a67aec032c8dddef0614df6419f has broken clang rebuilds:

comp-util.c:916:2: error: void function 'cal_comp_transfer_item_to' should not return a value [-Wreturn-type]
        g_return_val_if_fail (E_IS_CAL_CLIENT (src_client), FALSE);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
comp-util.c:917:2: error: void function 'cal_comp_transfer_item_to' should not return a value [-Wreturn-type]
        g_return_val_if_fail (E_IS_CAL_CLIENT (dest_client), FALSE);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/include/glib-2.0/glib/gmessages.h:354:3: note: expanded from macro 'g_return_val_if_fail'
         return (val);                                                  \
         ^      ~~~~~
comp-util.c:918:2: error: void function 'cal_comp_transfer_item_to' should not return a value [-Wreturn-type]
        g_return_val_if_fail (icalcomp_vcal != NULL, FALSE);
Comment 1 Fabiano Fidêncio 2013-11-11 12:19:45 UTC
Created attachment 259551 [details] [review]
Bug #711835 - Clang build in broken: void function 'cal_comp_transfer_item_to' should not return a value
Comment 2 Milan Crha 2013-11-11 12:30:36 UTC
Review of attachment 259551 [details] [review]:

not compiled, but looks correct, please commit to gnome-3-10 and master. Thanks.