Closed
Bug 660765
Opened 13 years ago
Closed 12 years ago
GCLI Requisition._onAssignmentChange could be simplified
Categories
(DevTools :: Console, defect, P3)
DevTools
Console
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 17
People
(Reporter: jwalker, Assigned: jwalker)
References
Details
It's possible that Requisition._onAssignmentChange could be simplified by having each Argument be aware of the input array/index that it belongs to - that way we could say to the Argument - adjust yourself, and we wouldn't need to do the search/alter operation that we do in _onAssignmentChange. To be aware of: - Some of the objects in the system are sort of 'final' in that we try not to change them post-creation. We should check this before we plough on - Arguments are used in 2 ways which might need tweaking - We _tokenize() the typed input into 'RawArgument's, and then _assign() these arguments to Assignments, which sometimes involves creation of higher-order Arguments like TrueNamedArgument and ArrayArgument which remember the Argument(s) from which they were created. HOWEVER we don't always go for higher-order Arguments. Perhaps we should formalize the distinction between the original Arguments from _tokenize() (Maybe called RawArgument) and the types of argument that we put into Assignments.
Assignee | ||
Updated•13 years ago
|
Blocks: GCLI-FUTURE
Assignee | ||
Comment 1•13 years ago
|
||
Moving GCLI bugs to Developer Tools: Console. Filter on 'baked beans are off'.
Component: Developer Tools → Developer Tools: Console
Assignee | ||
Updated•12 years ago
|
No longer blocks: GCLI-FUTURE
Assignee | ||
Comment 3•12 years ago
|
||
This bug should be solved automatically by bug 773565.
Assignee: nobody → jwalker
Status: NEW → ASSIGNED
Depends on: 773565
Target Milestone: --- → Firefox 17
Assignee | ||
Comment 4•12 years ago
|
||
Triage
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•