Closed
Bug 569973
Opened 14 years ago
Closed 14 years ago
More Dehydra/Spidermonkey problems with hasOwnProperty
Categories
(Developer Infrastructure :: Source Code Analysis, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ehren.m, Unassigned)
References
Details
Attachments
(4 files, 1 obsolete file)
I've been running SpiderMonkey from rev. 89e665eb9944 to avoid the test-suite failures in bug 526299 however I've just encountered a similar problem. Not sure if this is a SpiderMonkey issue or a "we're embedding SpiderMonkey wrong" issue. Output: /home/ehren/gcc4.5/dist.4.5.0/bin/g++ has.c -fplugin=/home/ehren/gcc4.5/dehydra/gcc_treehydra.so -fplugin-arg-gcc_treehydra-script=/home/ehren/dehydra-tests/testcode/../hasOwn.js /home/ehren/gcc4.5/dehydra/libs/treehydra.js:33: JS Exception: No blah in this lazy object :0: #0: Error("No blah in this lazy object") /home/ehren/gcc4.5/dehydra/libs/treehydra.js:33: #1: unhandledLazyProperty("blah") /home/ehren/dehydra-tests/testcode/../hasOwn.js:7: #2: process_tree([object GCCNode]) make: *** [blahmake] Error 1 (See attached backtrace and testcase)
Reporter | ||
Comment 1•14 years ago
|
||
Reporter | ||
Comment 2•14 years ago
|
||
Reporter | ||
Comment 3•14 years ago
|
||
can confirm that patch in bug 570195 fixes this.
Attachment #449340 -
Flags: review?(tglek)
Updated•14 years ago
|
Attachment #449340 -
Flags: review?(tglek) → review+
Comment 4•14 years ago
|
||
Comment on attachment 449340 [details] [diff] [review] for testsuite Should put the spidermonkey bugno in a comment in that file in case people run into it.
Reporter | ||
Comment 5•14 years ago
|
||
carrying over r+
Attachment #449340 -
Attachment is obsolete: true
Attachment #449342 -
Flags: review+
Reporter | ||
Comment 6•14 years ago
|
||
http://hg.mozilla.org/rewriting-and-analysis/dehydra/rev/225275b2e174
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•6 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
•