[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Chicken-hackers] [PATCH] Use noreturn attribute in clang versions w
|
From: |
Peter Bex |
|
Subject: |
Re: [Chicken-hackers] [PATCH] Use noreturn attribute in clang versions which support it |
|
Date: |
Wed, 23 Oct 2013 11:58:34 +0200 |
|
User-agent: |
Mutt/1.4.2.3i |
On Wed, Oct 23, 2013 at 11:53:13AM +0200, Moritz Heidkamp wrote:
> BTW, I got another warning when running the tests:
>
> ../chicken 'embedded4.scm' -output-file 'a.c' -scrutinize -verbose
> -include-path ..
> 'clang' 'a.c' -o 'a.o' -c -fno-strict-aliasing -fwrapv
> -DHAVE_CHICKEN_CONFIG_H -DC_ENABLE_PTABLES -g -Wall -Wno-unused -DC_EMBEDDED
> -I.. -I"/home/syn/code/public/chicken/core/target/include/chicken"
> 'clang' 'embedded3.c' -o 'embedded3.o' -c -fno-strict-aliasing -fwrapv
> -DHAVE_CHICKEN_CONFIG_H -DC_ENABLE_PTABLES -g -Wall -Wno-unused -DC_EMBEDDED
> -I.. -I"/home/syn/code/public/chicken/core/target/include/chicken"
> embedded3.c:23:26: warning: format specifies type 'unsigned int' but the
> argument has type 'long' [-Wformat]
> printf("data: %08x\n", val);
> ~~~~ ^~~
> %08lx
>
> Not sure if this is something to worry about, though.
Ah, I guess we should take a look at it. It's not a huge deal, but for
sanity we should use the correct format or cast it, or something.
Cheers,
Peter
--
http://www.more-magic.net