[apparmor] [PATCH] libapparmor: Fix -Wunused-but-set-variable GCC warning
Seth Arnold
seth.arnold at canonical.com
Sat Jan 30 00:00:30 UTC 2016
On Fri, Jan 29, 2016 at 05:09:09PM -0600, Tyler Hicks wrote:
> > > Signed-off-by: Tyler Hicks <tyhicks at canonical.com>
> > Did I miss something?
>
> No, it is confusing and I dug deep enough to figure out that it is
> working as intended. Here's what I know (requires building libapparmor):
>
> libraries/libapparmor/src/grammar.c:
> 62 /* Substitute the variable and function names. */
> 63 #define yyparse aalogparse_parse
Aha! I skipped this #define because it's "the wrong way around". I can't
recall seeing this trick before. Thanks for explaning it.
Acked-by: Seth Arnold <seth.arnold at canonical.com>
Thanks
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20160129/a1571d7c/attachment.pgp>
More information about the AppArmor
mailing list