diff mbox

ui/cocoa.m: eliminate normalWindow warning

Message ID 57057D6E-C108-4AE1-8370-E7E6855B2F2C@gmail.com
State New
Headers show

Commit Message

Programmingkid Sept. 25, 2015, 10:34 p.m. UTC
Eliminate this warning associated with the setting of the normalWindow's title:

ui/cocoa.m: In function '-[QemuCocoaAppController init]':
ui/cocoa.m:888:9: warning: format not a string literal and no format arguments
 [-Wformat-security]
         [normalWindow setTitle:[NSString stringWithFormat:@"QEMU"]];


Signed-off-by: John Arbuckle <programmingkidx@gmail.com>

---
 ui/cocoa.m |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Peter Maydell Oct. 13, 2015, 8:33 p.m. UTC | #1
On 25 September 2015 at 23:34, Programmingkid <programmingkidx@gmail.com> wrote:
> Eliminate this warning associated with the setting of the normalWindow's title:
>
> ui/cocoa.m: In function '-[QemuCocoaAppController init]':
> ui/cocoa.m:888:9: warning: format not a string literal and no format arguments
>  [-Wformat-security]
>          [normalWindow setTitle:[NSString stringWithFormat:@"QEMU"]];
>
>
> Signed-off-by: John Arbuckle <programmingkidx@gmail.com>
>
> ---
>  ui/cocoa.m |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/ui/cocoa.m b/ui/cocoa.m
> index 334e6f6..fa21fdb 100644
> --- a/ui/cocoa.m
> +++ b/ui/cocoa.m
> @@ -855,7 +855,7 @@ QemuCocoaView *cocoaView;
>              exit(1);
>          }
>          [normalWindow setAcceptsMouseMovedEvents:YES];
> -        [normalWindow setTitle:[NSString stringWithFormat:@"QEMU"]];
> +        [normalWindow setTitle:@"QEMU"];
>          [normalWindow setContentView:cocoaView];
>  #if (MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_X_VERSION_10_10)
>          [normalWindow useOptimizedDrawing:YES];
> --
> 1.7.5.4

Thanks, applied to cocoa.next (sorry it's taken me a while
to get back to these).

-- PMM
diff mbox

Patch

diff --git a/ui/cocoa.m b/ui/cocoa.m
index 334e6f6..fa21fdb 100644
--- a/ui/cocoa.m
+++ b/ui/cocoa.m
@@ -855,7 +855,7 @@  QemuCocoaView *cocoaView;
             exit(1);
         }
         [normalWindow setAcceptsMouseMovedEvents:YES];
-        [normalWindow setTitle:[NSString stringWithFormat:@"QEMU"]];
+        [normalWindow setTitle:@"QEMU"];
         [normalWindow setContentView:cocoaView];
 #if (MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_X_VERSION_10_10)
         [normalWindow useOptimizedDrawing:YES];