Closed
Bug 438838
Opened 17 years ago
Closed 17 years ago
dehydra: constructors of static variables don't have .fieldOf
Categories
(Developer Infrastructure :: Source Code Analysis, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: benjamin, Assigned: taras.mozilla)
Details
Attachments
(3 files)
143 bytes,
text/plain
|
Details | |
5.85 KB,
text/plain
|
Details | |
4.49 KB,
patch
|
Details | Diff | Splinter Review |
Testcase attached: I expect every call to a constructor to have a .fieldOf, but in this case the constructor is being called on a static variable and doesn't have a .fieldOf.
Reporter | ||
Comment 1•17 years ago
|
||
Testcase .js
Assignee | ||
Comment 2•17 years ago
|
||
gcc init stuff is weird. Basic problem here is that dehydra does not apply a transformation pass to the C++ ast to make it into a simpler form(like the simpler form provided by treehydra)...because that'd be too much work...the downside is that code gets really messy when we try to stick nested c++ expressions that are in the AST into a flat list where variables are defined on the toplevel.
Additionally, having a couple of different kinds of node initialization in GCC does not help. In this particular case code for handling DECL_EXPR and INIT_EXPR wasn't unified enough.
Assignee: nobody → tglek
Status: NEW → ASSIGNED
Reporter | ||
Comment 3•17 years ago
|
||
This WFM.
Assignee | ||
Comment 4•17 years ago
|
||
pushed
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•7 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
•