Closed
Bug 1098740
Opened 11 years ago
Closed 11 years ago
Add a spawn function to sdk/event/utils
Categories
(Add-on SDK Graveyard :: General, defect, P2)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: evold, Assigned: evold)
References
Details
Attachments
(1 file)
Taken from bug 854982 see https://github.com/mozilla/addon-sdk/pull/1595/files
| Assignee | ||
Updated•11 years ago
|
Assignee: nobody → evold
| Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8522673 -
Flags: review?(rFobic)
Updated•11 years ago
|
Priority: -- → P2
Comment 2•11 years ago
|
||
Comment on attachment 8522673 [details] [review]
Link to Github pull-request: https://github.com/mozilla/addon-sdk/pull/1713
Hey I'm sorry to reject this but now `sdk/event/utils` have `Reactor` https://github.com/mozilla/addon-sdk/blob/master/lib/sdk/event/utils.js#L234-L257 that does more or less same, I don't think it would make sense to add spawn now, we should probably just update uses of `spawn` to use `Reactor` instead.
Attachment #8522673 -
Flags: review?(rFobic) → review-
| Assignee | ||
Comment 3•11 years ago
|
||
ah alright that's fine.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•