Closed Bug 712168 Opened 14 years ago Closed 2 years ago

Change BindData+Binder into an abstract base class

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Version:
Other Branch
Type:
defect

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: jorendorff, Unassigned)

Details

Currently, struct BindData has an anonymous union with only one element. That is silly. This should be an abstract base class with one virtual method, and two derived classes.
BindData goes away in the destructuring patches I have, although the exact design of how that goes may be up for debate, of course. So just making the minimal refactoring might not be helpful to me. I dunno, I haven't tried looking at/rebasing into the let patch at all yet...
Assignee: general → nobody
Severity: normal → S3

struct BindData no longer exists.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.