Closed
Bug 1257320
Opened 9 years ago
Closed 9 years ago
icu\source\common\unicode/ucasemap.h(93): warning C4577: 'noexcept' used with no exception handling mode specified; termination on exception is not guaranteed. Specify /EHsc from unified dns
Categories
(Core :: Networking: DNS, defect)
Core
Networking: DNS
Tracking
()
RESOLVED
FIXED
mozilla48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: gps, Assigned: gps)
References
Details
(Whiteboard: [necko-active])
Attachments
(1 file)
This is similar to bug 1257040 and bug 1256513 and a few other bugs. We're unmasking this warning and it gets turned into an error when building with Visual Studio 2015 Update 1 in automation.
This warning has been masked in a number of moz.build files. I'll file a follow-up to deal with it more robustly.
Assignee | ||
Comment 1•9 years ago
|
||
I did not mean bug 1256513. Look at the blockers for bug 1124033 for similar bug summaries as this one.
Assignee | ||
Comment 2•9 years ago
|
||
As part of unblocking building with VS2015u1 in automation, I'm mass
disabling compiler warnings that are turned into errors. This is not
the preferred mechanism to fix compilation warnings. So hopefully
this patch never lands because someone insists of fixing the underlying
problem instead. But if it does land, hopefully the workaround is
only temporary. That being said, the warnings this patch masks are
in an included ICU header, which is a 3rd party project. So our hands
appear tied.
Review commit: https://reviewboard.mozilla.org/r/40569/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/40569/
Attachment #8731416 -
Flags: review?(honzab.moz)
![]() |
||
Comment 3•9 years ago
|
||
Same here as in bug 1256473.
![]() |
||
Comment 4•9 years ago
|
||
Comment on attachment 8731416 [details]
MozReview Request: Bug 1257320 - Disable C4577 to unblock compilation on VS2015; r?mayhemer
https://reviewboard.mozilla.org/r/40569/#review37561
Attachment #8731416 -
Flags: review?(honzab.moz) → review+
Comment 6•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•