The default bug view has changed. See this FAQ.

Disable MSVC warning C4482: nonstandard extension used: enum 'xyz' used in qualified name

RESOLVED FIXED in mozilla21

Status

()

Core
Build Config
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: filidautore, Assigned: filidautore)

Tracking

unspecified
mozilla21
x86
Windows 7
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

4 years ago
It becomes a standard since C++11, and MSVC11 removed the warning.
(See bug 755031 comment 23 and 24)
(Assignee)

Comment 1

4 years ago
Created attachment 703869 [details] [diff] [review]
Disable globally MSVC warning C4482
Attachment #703869 - Flags: review?(ted)
Assignee: nobody → filidautore
Comment on attachment 703869 [details] [diff] [review]
Disable globally MSVC warning C4482

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

::: configure.in
@@ +2201,5 @@
>          # MSVC warning C4345 warns of newly conformant behavior as of VS2003.
>          # MSVC warning C4351 warns of newly conformant behavior as of VS2005.
> +        # MSVC warning C4482 warns when an enum value is refered specifing the
> +        # name of the enum itself.  It becomes a standard since C++11, and MSVC11
> +        # removed the warning.

Can you reword this slightly, perhaps "This behavior is allowed in C++11, and the warning has been removed in VS2012."
Attachment #703869 - Flags: review?(ted) → review+
If you can upload a patch with that minor rewording, you can add the "checkin-needed" keyword to this bug and we'll get it landed. Thanks for the patch!
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
(Assignee)

Comment 4

4 years ago
Created attachment 703890 [details] [diff] [review]
Disable globally MSVC warning C4482 - v1.1

Updated the comment in the patch as suggested.
Attachment #703869 - Attachment is obsolete: true
Attachment #703890 - Flags: review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
(Assignee)

Comment 5

4 years ago
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #3)
Thanks to you for fast review :)
Thanks for proofing my poor English :)
https://hg.mozilla.org/integration/mozilla-inbound/rev/8472e5898021
Flags: in-testsuite-
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/8472e5898021
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Doing this still causes build errors for people, no?
What build errors?
jgriffin had a build issue because of a gfxImageFormat:: prefix on mac on Jan 8, but the pastebin is dead already.
Obviously this patch will affect only MSVC.
A better fix would be making gfxImageFormat an enum class as dholbert said.
You need to log in before you can comment on or make changes to this bug.