Bug#742214: libskk: fails to build with clang instead of gcc

[Nicolas Sévelin-Radiguet]

Source: libskk
Version: 1.0.0-1
Severity: minor
Tags: patch
User: pkg-llvm-t...@lists.alioth.debian.org
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/libskk

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 <nic...@free.fr>
Last-Update: 2014-03-20

--- a/libskk/state.vala
+++ b/libskk/state.vala
@@ -269,7 +269,7 @@
                         break;
                     }
                 } catch (GLib.RegexError e) {
-                    return_if_reached ();
+                    return_val_if_reached (null);
                 }
                             
                 int match_start_pos, match_end_pos;
--- a/libskk/expr.vala
+++ b/libskk/expr.vala
@@ -59,8 +59,7 @@
         }
 
         public ExprNode read_string (string expr, ref int index) {
-            return_val_if_fail (index < expr.length && expr[index] == '"',
-                                null);
+            return_if_fail (index < expr.length && expr[index] == '"');
             var builder = new StringBuilder ();
             index++;
             bool stop = false;