Closed
Bug 271814
Opened 20 years ago
Closed 20 years ago
build bustage VS6 in nsSVGUseElement.cpp
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
People
(Reporter: daa, Assigned: jwatt)
Details
Attachments
(1 file)
|
1.65 KB,
patch
|
Callek
:
review+
|
Details | Diff | Splinter Review |
building under MS VS6, build fails with : d:/mozsvg/mozilla/content/svg/content/src/nsSVGUseElement.cpp(524) : error C2374 : 'i' : redefinition; multiple initialization d:/mozsvg/mozilla/content/svg/content/src/nsSVGUseElement.cpp(509) : see declaration of 'i' looks like VS6 does not like args in "for" defined in the statement
|
Assignee | |
Comment 1•20 years ago
|
||
Assignee: tor → jonathan.watt
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•20 years ago
|
Severity: normal → critical
|
||
Comment 2•20 years ago
|
||
Comment on attachment 167084 [details] [diff] [review] patch r+ on this trivial fix, follows our code guidelines to fix that for-loop. I am not a peer of course, though was around IRC and will try and get someone to comit.
Attachment #167084 -
Flags: review+
|
|
||
Comment 3•20 years ago
|
||
timeless comitted
Severity: critical → normal
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 4•20 years ago
|
||
You knocked the severity to normal when you made that last comment, returning to critical. :)
Severity: normal → critical
You need to log in
before you can comment on or make changes to this bug.
Description
•