Closed
Bug 1411056
Opened 7 years ago
Closed 7 years ago
SVGSVGElement: unnecessary parentheses in declaration of 'NS_NewSVGSVGElement'
Categories
(Developer Infrastructure :: Source Code Analysis, defect)
Developer Infrastructure
Source Code Analysis
Tracking
(firefox58 affected)
RESOLVED
INVALID
Tracking | Status | |
---|---|---|
firefox58 | --- | affected |
People
(Reporter: Sylvestre, Assigned: Sylvestre)
References
()
Details
Attachments
(1 file)
In file included from /root/firefox-gcc-last/image/SVGDocumentWrapper.cpp:27:0,
from /root/firefox-gcc-last/obj-x86_64-pc-linux-gnu/image/Unified_cpp_image1.cpp:38:
/root/firefox-gcc-last/obj-x86_64-pc-linux-gnu/dist/include/mozilla/dom/SVGSVGElement.h:85:19: error: unnecessary parentheses in declaration of 'NS_NewSVGSVGElement' [-Werror=parentheses]
friend nsresult (::NS_NewSVGSVGElement(nsIContent **aResult,
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8921237 [details]
Bug 1411056 - SVGSVGElement: Remove unnecessary parentheses in declaration of 'NS_NewSVGSVGElement'
https://reviewboard.mozilla.org/r/192244/#review197426
r=me
Attachment #8921237 -
Flags: review?(dholbert) → review+
Updated•7 years ago
|
Assignee: nobody → sledru
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/be03517460fd
SVGSVGElement: Remove unnecessary parentheses in declaration of 'NS_NewSVGSVGElement' r=dholbert
Comment 4•7 years ago
|
||
Backed out for build bustage at dom/SVGSVGElement.h:87:
https://hg.mozilla.org/integration/autoland/rev/01c28e18b63c00989775445dbeec6adc697b17b7
Push with bustage: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=be03517460fdd4a409ef6b01669f861a27aa1d3b&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=runnable
Build log: https://treeherder.mozilla.org/logviewer.html#?job_id=139100911&repo=autoland
[task 2017-10-24T06:14:15.446Z] 06:14:15 INFO - In file included from /builds/worker/workspace/build/src/image/SVGDocumentWrapper.cpp:27:0,
[task 2017-10-24T06:14:15.446Z] 06:14:15 INFO - from /builds/worker/workspace/build/src/obj-firefox/image/Unified_cpp_image1.cpp:38:
[task 2017-10-24T06:14:15.446Z] 06:14:15 INFO - /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/dom/SVGSVGElement.h:87:77: error: 'enum class nsresult' is not a class or a namespace
[task 2017-10-24T06:14:15.446Z] 06:14:15 INFO - mozilla::dom::FromParser aFromParser);
[task 2017-10-24T06:14:15.446Z] 06:14:15 INFO - ^
[task 2017-10-24T06:14:15.446Z] 06:14:15 INFO - /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/dom/SVGSVGElement.h:87:77: error: ISO C++ forbids declaration of 'NS_NewSVGSVGElement' with no type [-fpermissive]
[task 2017-10-24T06:14:15.446Z] 06:14:15 INFO - /builds/worker/workspace/build/src/config/rules.mk:1072: recipe for target 'Unified_cpp_image1.o' failed
[task 2017-10-24T06:14:15.446Z] 06:14:15 INFO - gmake[5]: *** [Unified_cpp_image1.o] Error 1
Flags: needinfo?(sledru)
Comment 5•7 years ago
|
||
This should not land since it's a bug in gcc. I've forwarded a gcc bug on this matter.
Updated•7 years ago
|
Comment 6•7 years ago
|
||
As pe comment: https://bugzilla.mozilla.org/show_bug.cgi?id=1411056#c5
Flags: needinfo?(sledru)
Comment 7•7 years ago
|
||
Thanks! Calling this INVALID then, since it's not actually a bug in Mozilla code.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Updated•7 years ago
|
Product: Core → Firefox Build System
Updated•2 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•