Closed
Bug 623117
Opened 15 years ago
Closed 15 years ago
JetPack class Handle depends on invalid unqualified lookup into dependent bases of class templates
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla2.0b10
People
(Reporter: espindola, Assigned: espindola)
References
Details
Attachments
(1 file)
662 bytes,
patch
|
mrbkap
:
review+
benjamin
:
approval2.0+
|
Details | Diff | Splinter Review |
js/jetpack/Handle.h expects to be able to access dependent base classes
members without adding a prefix.
The issue is explained in http://clang.llvm.org/compatibility.html#c++
Assignee | ||
Comment 1•15 years ago
|
||
Attachment #501233 -
Flags: review?(mrbkap)
Assignee | ||
Updated•15 years ago
|
Blocks: clang-macosx
Assignee | ||
Updated•15 years ago
|
Summary: epends on invalid unqualified lookup into dependent bases of class templates → Depends on invalid unqualified lookup into dependent bases of class templates
The reporter's summary and initial comment were both lame. I'm merely adjusting the summary and providing a better link. I am not passing judgement on the quality of the bug report.
http://clang.llvm.org/compatibility.html#dep_lookup_bases
OS: Mac OS X → Windows 7
Summary: Depends on invalid unqualified lookup into dependent bases of class templates → JetPack class Handle depends on invalid unqualified lookup into dependent bases of class templates
Assignee | ||
Comment 3•15 years ago
|
||
I am sure I was not using windows.
OS: Windows 7 → Mac OS X
Hardware: x86 → x86_64
Updated•15 years ago
|
Attachment #501233 -
Flags: review?(mrbkap) → review+
Assignee | ||
Updated•15 years ago
|
Attachment #501233 -
Flags: approval2.0?
![]() |
||
Comment 4•15 years ago
|
||
Thanks for reporting this, Rafael.
Updated•15 years ago
|
Attachment #501233 -
Flags: approval2.0? → approval2.0+
Assignee | ||
Updated•15 years ago
|
Keywords: checkin-needed
Comment 5•15 years ago
|
||
Assignee: general → respindola
Status: NEW → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b10
You need to log in
before you can comment on or make changes to this bug.
Description
•