Closed
Bug 576363
Opened 15 years ago
Closed 15 years ago
Qualify accesses to dependent base in js
Categories
(Core Graveyard :: Nanojit, defect)
Core Graveyard
Nanojit
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sharparrow1, Unassigned)
Details
(Whiteboard: fixed-in-nanojit, fixed-in-tracemonkey, fixed-in-tamarin)
Attachments
(1 file)
|
603 bytes,
patch
|
n.nethercote
:
review+
|
Details | Diff | Splinter Review |
Patch attached. Similar to bug 573210, but in js/src/. See bug 573210 comment 6 for an explanation of why clang isn't accepting code gcc does accept.
Attachment #455539 -
Flags: review?(brendan)
Comment 1•15 years ago
|
||
Note different module (and repo -- nanojit-central) from JS engine proper.
/be
Assignee: sharparrow1 → nobody
Component: JavaScript Engine → Nanojit
OS: Linux → All
QA Contact: general → nanojit
Hardware: x86 → All
Updated•15 years ago
|
Attachment #455539 -
Flags: review?(brendan) → review?(nnethercote)
Comment 2•15 years ago
|
||
Comment on attachment 455539 [details] [diff] [review]
Patch
The change appears fine, I'll take your word that it fixes the relevant problem when compiling with clang.
Eli, do you want me to land this patch for you?
Attachment #455539 -
Flags: review?(nnethercote) → review+
| Reporter | ||
Comment 3•15 years ago
|
||
Yes, please.(In reply to comment #2)
> Eli, do you want me to land this patch for you?
Yes, please.
Comment 4•15 years ago
|
||
Whiteboard: fixed-in-nanojit
Comment 5•15 years ago
|
||
Whiteboard: fixed-in-nanojit → fixed-in-nanojit, fixed-in-tracemonkey
Comment 6•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment 7•15 years ago
|
||
Whiteboard: fixed-in-nanojit, fixed-in-tracemonkey → fixed-in-nanojit, fixed-in-tracemonkey, fixed-in-tamarin
| Assignee | ||
Updated•11 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•