|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
58 bytes, text/x-review-board-request
|Details | Review|
We're seeing this warning turned into an error when building layout/forms unified sources with VS2015u1 in automation.
Created attachment 8730474 [details] MozReview Request: Bug 1256509 - Disable C4577 to unblock compilation on VS2015; r?poiru 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 warning occurs in 3rd party ICU code. So I'm not sure what my options are for fixing this. Review commit: https://reviewboard.mozilla.org/r/39913/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/39913/
Attachment #8730474 - Flags: review?(birunthan)
Comment on attachment 8730474 [details] MozReview Request: Bug 1256509 - Disable C4577 to unblock compilation on VS2015; r?poiru https://reviewboard.mozilla.org/r/39913/#review36493
Attachment #8730474 - Flags: review?(birunthan) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 2 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.