[prev in list] [next in list] [prev in thread] [next in thread] 

List:       apparmor-dev
Subject:    Re: [apparmor] Fwd: Bug#756807: apparmor: FTBFS with clang instead of gcc
From:       Steve Beattie <steve () nxnw ! org>
Date:       2014-08-29 20:24:10
Message-ID: 20140829202410.GU5845 () nxnw ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Fri, Aug 29, 2014 at 12:15:29PM -0700, Seth Arnold wrote:
> This patch adds 'static' to all inlined functions in the
> parser_interface.c file to address Debian bug 756807.
> 
> The parser still passed 'make check' when compiled with gcc. I haven't
> compiled the package with clang to ensure that this is sufficient.
> 
> Signed-off-by: Seth Arnold <seth.arnold@canonical.com>
Acked-by: Steve Beattie <steve@nxnw.org>

Though the change is less of an issue for trunk, presumably because the
compiler is invoked with the gnu++0x standard. It does let clang warn
that sd_write_uint8, sd_write_blob, sd_write_list, and sd_write_listend
are all unused functions currently.

-- 
Steve Beattie
<sbeattie@ubuntu.com>
http://NxNW.org/~steve/

["signature.asc" (application/pgp-signature)]

-- 
AppArmor mailing list
AppArmor@lists.ubuntu.com
Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor


[prev in list] [next in list] [prev in thread] [next in thread]