Closed
Bug 1257040
Opened 8 years ago
Closed 8 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
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: gps, Assigned: gps)
References
Details
Attachments
(1 file)
This is similar to bug 1256509 but for code in js/src.
Assignee | ||
Comment 1•8 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/40309/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/40309/
Attachment #8731011 -
Flags: review?(jorendorff)
Updated•8 years ago
|
Attachment #8731011 -
Flags: review?(jorendorff) → review+
Comment 2•8 years ago
|
||
Comment on attachment 8731011 [details] MozReview Request: Bug 1257040 - Disable C4577 to unblock compilation on VS2015; r?jorendorff https://reviewboard.mozilla.org/r/40309/#review36935 This warning is irrelevant to us anyway.
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → gps
Status: NEW → ASSIGNED
Comment 4•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f3d0a6754976
Status: ASSIGNED → RESOLVED
Closed: 8 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
•