Closed
Bug 1257346
Opened 8 years ago
Closed 3 years ago
dom/plugins/ipc/PluginInstanceChild.cpp(93): warning C4312: 'type cast': conversion from 'unsigned int' to 'HIMC' of greater size
Categories
(Core Graveyard :: Plug-ins, defect, P5)
Core Graveyard
Plug-ins
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: gps, Unassigned)
References
Details
(Keywords: good-first-bug)
Attachments
(1 file)
This warning is about to be unmasked in bug 1124033. It is turned into an error when building with Visual Studio 2015 Update 1 in automation.
Reporter | ||
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 on fixing the underlying problem instead. But if it does land, hopefully the workaround is only temporary. Review commit: https://reviewboard.mozilla.org/r/40613/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/40613/
Attachment #8731460 -
Flags: review?(aklotz)
Reporter | ||
Comment 2•8 years ago
|
||
Comment on attachment 8731460 [details] MozReview Request: Bug 1257346 - Disable C4312 to unblock compilation on VS2015; r?aklotz Review request updated; see interdiff: https://reviewboard.mozilla.org/r/40613/diff/1-2/
Updated•8 years ago
|
Attachment #8731460 -
Flags: review?(aklotz) → review+
Comment 3•8 years ago
|
||
Comment on attachment 8731460 [details] MozReview Request: Bug 1257346 - Disable C4312 to unblock compilation on VS2015; r?aklotz https://reviewboard.mozilla.org/r/40613/#review37935
Comment 5•8 years ago
|
||
Backed out for Windows build bustage. Backout: https://hg.mozilla.org/integration/mozilla-inbound/rev/272ec90eff71 Push with failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=f17a9b548316 Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=24265176&repo=mozilla-inbound 12:24:53 INFO - ============================== 12:24:53 INFO - ERROR PROCESSING MOZBUILD FILE 12:24:53 INFO - ============================== 12:24:53 INFO - The error occurred while processing the following file: 12:24:53 INFO - c:/builds/moz2_slave/m-in-w64-d-0000000000000000000/build/src/dom/plugins/ipc/moz.build 12:24:53 INFO - The error was triggered on line 158 of this file: 12:24:53 INFO - SOURCES['PluginScriptableObjectChild.cpp'].flags += ['-wd4312'] 12:24:53 INFO - An error was encountered as part of executing the file itself. The error appears to be the fault of the script. 12:24:53 INFO - The error as reported by Python is: 12:24:53 INFO - ['KeyError: u"\'PluginScriptableObjectChild.cpp\'"\n'] 12:24:53 INFO - Makefile:124: recipe for target 'backend.RecursiveMakeBackend' failed Seems like PluginScriptableObjectChild.cpp needs to be added to https://dxr.mozilla.org/mozilla-central/source/dom/plugins/ipc/moz.build#97
Flags: needinfo?(gps)
Reporter | ||
Comment 6•8 years ago
|
||
Yeah, PluginScriptableObjectChild.cpp is in UNIFIED_SOURCES. Derp. The fix is to globally add -wd4312 for the entire directory. I'll do that and land.
Flags: needinfo?(gps)
Reporter | ||
Comment 7•8 years ago
|
||
I'm landing the -wd4312 in bug 1124033. Morphing this bug to track removing the disabling of the warning.
Updated•7 years ago
|
Updated•7 years ago
|
Mentor: benjamin
Comment 8•4 years ago
|
||
Hello, is this still an issue? I would like to work on this bug. How exactly do I reproduce the issue?
Flags: needinfo?(jmathies)
Updated•4 years ago
|
Severity: normal → S4
Flags: needinfo?(jmathies)
Priority: P3 → P5
Comment 9•3 years ago
|
||
Hello,
I am new to opensource community and I would like to contribute to this task as my first contribution, how can I start?
Thanks a lot!
Flags: needinfo?(jmathies)
Comment 10•3 years ago
|
||
Sorry, this is in the old plugins code, which is about to get ripped out. Nothing to do here.
Status: NEW → RESOLVED
Closed: 3 years ago
Flags: needinfo?(jmathies)
Resolution: --- → WONTFIX
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•