<div dir="ltr">Yes that would be much better.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Aug 10, 2014 at 4:50 AM, أحمد المحمودي <span dir="ltr"><<a href="mailto:aelmahmoudy@users.sourceforge.net" target="_blank">aelmahmoudy@users.sourceforge.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Aug 07, 2014 at 01:35:34PM -0500, Arthur Marble wrote:<br>
> diff -Naur covered.orig/covered-0.7.10/debian/patches/clang-ftbfs.diff covered/covered-0.7.10/debian/patches/clang-ftbfs.diff<br>
> --- covered.orig/covered-0.7.10/debian/patches/clang-ftbfs.diff       1969-12-31 18:00:00.000000000 -0600<br>
> +++ covered/covered-0.7.10/debian/patches/clang-ftbfs.diff    2014-08-07 13:19:49.684388142 -0500<br>
> @@ -0,0 +1,16 @@<br>
> +--- a/src/lxt2_read.h<br>
> ++++ b/src/lxt2_read.h<br>
> +@@ -47,10 +47,10 @@ typedef long off_t;<br>
> + #include <zlib.h><br>
> + #endif<br>
> +<br>
> +-#ifdef __GNUC__<br>
> +-#define _LXT2_RD_INLINE inline<br>
> +-#else<br>
> ++#ifdef __clang__<br>
> + #define _LXT2_RD_INLINE<br>
> ++#else<br>
> ++#define _LXT2_RD_INLINE inline<br>
> + #endif<br>
> +<br>
> + #define LXT2_RDLOAD "LXTLOAD | "<br>
---end quoted text---<br>
<br>
  Shouldn't this better be:<br>
<br>
  #ifdef __clang__<br>
  #define _LXT2_RD_INLINE<br>
  #else<br>
  # ifdef __GNUC__<br>
  # define _LXT2_RD_INLINE inline<br>
  # else<br>
  # define _LXT2_RD_INLINE<br>
  # endif<br>
  #endif<br>
<br>
  in order to keep upstream logic the same ?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
 ‎أحمد المحمودي (Ahmed El-Mahmoudy)<br>
  Digital design engineer<br>
 GPG KeyID: 0xEDDDA1B7<br>
 GPG Fingerprint: 8206 A196 2084 7E6D 0DF8  B176 BC19 6A94 EDDD A1B7<br>
</font></span></blockquote></div><br></div>