qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] Re: [PATCH] HACKING: Update status of format checking
       [not found] <1295890808-5447-1-git-send-email-weil@mail.berlios.de>
@ 2011-01-24 18:07 ` Stefan Weil
  0 siblings, 0 replies; only message in thread
From: Stefan Weil @ 2011-01-24 18:07 UTC (permalink / raw)
  To: QEMU Developers

Am 24.01.2011 18:40, schrieb Stefan Weil:
> Hopefully all functions with printf like arguments now use format checking.
>
> This was tested with default build configuration on linux
> and windows hosts (including some cross compilations),
> so chances are good that there remain few (if any) functions
> without format checking.
>
> Therefore the last comment in HACKING is no longer valid but misleading.
>
> Cc: Blue Swirl<blauwirbel@gmail.com>
> Signed-off-by: Stefan Weil<weil@mail.berlios.de>
> ---
>   HACKING |    3 ---
>   1 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/HACKING b/HACKING
> index 6ba9d7e..3af53fd 100644
> --- a/HACKING
> +++ b/HACKING
> @@ -120,6 +120,3 @@ gcc's printf attribute directive in the prototype.
>   This makes it so gcc's -Wformat and -Wformat-security options can do
>   their jobs and cross-check format strings with the number and types
>   of arguments.
> -
> -Currently many functions in QEMU are not following this rule but
> -patches to add the attribute would be very much appreciated.
>    

Sorry, I did not send this patch to qemu-devel for review.

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2011-01-24 18:07 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1295890808-5447-1-git-send-email-weil@mail.berlios.de>
2011-01-24 18:07 ` [Qemu-devel] Re: [PATCH] HACKING: Update status of format checking Stefan Weil

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).