* [Qemu-devel] [PATCH, RFC] gcc -Wredundant-decls
@ 2008-08-23 15:47 Blue Swirl
2008-09-07 2:21 ` Anthony Liguori
0 siblings, 1 reply; 2+ messages in thread
From: Blue Swirl @ 2008-08-23 15:47 UTC (permalink / raw)
To: qemu-devel
[-- Attachment #1: Type: text/plain, Size: 263 bytes --]
The first patch enables gcc flag -Wredundant-decls, the second one
fixes the warnings caused by it. Again, with the flag enabled, dyngen
targets output a lot of useless warnings, so it should be enabled
later. Applying the fixing patch should be safe.
Comments?
[-- Attachment #2: wredundant_decls.diff --]
[-- Type: plain/text, Size: 1005 bytes --]
[-- Attachment #3: fix_wredundant_decls.diff --]
[-- Type: plain/text, Size: 22823 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [Qemu-devel] [PATCH, RFC] gcc -Wredundant-decls
2008-08-23 15:47 [Qemu-devel] [PATCH, RFC] gcc -Wredundant-decls Blue Swirl
@ 2008-09-07 2:21 ` Anthony Liguori
0 siblings, 0 replies; 2+ messages in thread
From: Anthony Liguori @ 2008-09-07 2:21 UTC (permalink / raw)
To: qemu-devel
Blue Swirl wrote:
> The first patch enables gcc flag -Wredundant-decls, the second one
> fixes the warnings caused by it. Again, with the flag enabled, dyngen
> targets output a lot of useless warnings, so it should be enabled
> later. Applying the fixing patch should be safe.
>
You may have applied them already, but they both look sane to me.
Regards,
Anthony Liguori
> Comments?
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-09-07 2:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-08-23 15:47 [Qemu-devel] [PATCH, RFC] gcc -Wredundant-decls Blue Swirl
2008-09-07 2:21 ` Anthony Liguori
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).