Source: nemo
Version: 1.8.4-1.1
Severity: minor
Tags: patch
User: [email protected]
Usertags: clang-ftbfs

Dear Maintainer,

Your package fails to build with clang instead of gcc. [-Wreturn-type]
The attached patch fixes it.
Buildlogs and patch are here:
https://github.com/nonas/debian-clang/tree/master/buildlogs/nemo

Regards,
Nicolas

-- System Information:
Debian Release: jessie/sid
  APT prefers testing-updates
  APT policy: (500, 'testing-updates'), (500, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.13-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Description: fix FTBFS with clang instead of gcc
Author: Nicolas Sévelin-Radiguet <[email protected]>
Last-Update: 2014-03-16
--- a/src/nemo-icon-view.c
+++ b/src/nemo-icon-view.c
@@ -1197,7 +1197,7 @@
 static NemoZoomLevel
 nemo_icon_view_get_default_zoom_level (NemoView *view)
 {
-    g_return_if_fail (NEMO_IS_ICON_VIEW (view));
+    g_return_val_if_fail (NEMO_IS_ICON_VIEW (view), FALSE);
 
     return get_default_zoom_level(NEMO_ICON_VIEW (view));
 }
--- a/src/nemo-list-view.c
+++ b/src/nemo-list-view.c
@@ -2852,7 +2852,7 @@
 static NemoZoomLevel
 nemo_list_view_get_default_zoom_level (NemoView *view)
 {
-    g_return_if_fail (NEMO_IS_LIST_VIEW (view));
+    g_return_val_if_fail (NEMO_IS_LIST_VIEW (view), FALSE);
 
     return get_default_zoom_level();
 }
--- a/src/nemo-view.c
+++ b/src/nemo-view.c
@@ -683,7 +683,7 @@
 NemoZoomLevel
 nemo_view_get_default_zoom_level (NemoView *view)
 {
-    g_return_if_fail (NEMO_IS_VIEW (view));
+    g_return_val_if_fail (NEMO_IS_VIEW (view), FALSE);
 
     if (!nemo_view_supports_zooming (view)) {
         return -1;

Reply via email to