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)

defect
Not set
normal

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 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+
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
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)
This should not land since it's a bug in gcc. I've forwarded a gcc bug on this matter.
Flags: needinfo?(sledru)
Thanks! Calling this INVALID then, since it's not actually a bug in Mozilla code.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Product: Core → Firefox Build System
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: