883721 - Fix -Wunused-private-field and -Wunused-variable warnings in widget/cocoa

Status: RESOLVED DUPLICATE of bug 847709

(Core :: Widget: Cocoa, defect, P4)

Description

[Chris Peterson [:cpeterson]]

Attachment: fix-cocoa-widget-warnings.patch

These are the only warnings I see in widget/cocoa/ code when building Firefox for OSX:

widget/cocoa/GfxInfo.h:82:12 [-Wunused-private-field] private field 'mRendererIDs' is not used
widget/cocoa/nsMacWebAppUtils.mm:49:8 [-Wunused-variable] unused variable 'success'

Comment 1

[Josh Aas]

Comment on attachment 763388 [details] [diff] [review]
fix-cocoa-widget-warnings.patch

Review of attachment 763388 [details] [diff] [review]:
-----------------------------------------------------------------

::: widget/cocoa/GfxInfo.h
@@ -78,5 @@
>  
>    nsString mAdapterVendorID;
>    nsString mAdapterDeviceID;
>  
> -  uint32_t mRendererIDs[16];

This is "used," it's just macro-d out. See:

http://mxr.mozilla.org/mozilla-central/search?string=mRendererIDs

Please either use ifdef 0 to turn it off like in the impl code, or, depending on why the impl is turned off (I don't know), also delete the impl code.

::: widget/cocoa/nsMacWebAppUtils.mm
@@ -49,5 @@
> -  BOOL success = [[NSWorkspace sharedWorkspace] launchAppWithBundleIdentifier:
> -                        [NSString stringWithCharacters:((nsString)bundleIdentifier).get() length:((nsString)bundleIdentifier).Length()]
> -                        options: (NSWorkspaceLaunchOptions)0
> -                        additionalEventParamDescriptor: nil
> -                        launchIdentifier: NULL];

I already have a fix for this. See bug 869083.

Comment 2

[Chris Peterson [:cpeterson]]

Thanks, Josh. I see the mRendererIDs portion of this bug is a duplicate of bug 847709.