Closed Bug 551365 Opened 15 years ago Closed 15 years ago

Review the byte streams module, jetpack-core/lib/byte-streams.js

Categories

(Add-on SDK Graveyard :: General, defect)

Product:
Add-on SDK Graveyard
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: adw, Unassigned)

References

Details

http://hg.mozilla.org/labs/jetpack-sdk/file/tip/packages/jetpack-core/lib/byte-streams.js Module description from the docs: This module contains functionality for byte input and output streams.
Wait, didn't I actually r+ this from bug 541622? Er, it looks like I didn't formally r+ it... In any case, I'm willing to take this one on if you want, since I'm already reviewing text-streams and the code is similar, and I didn't write it in the first place. :) Alternatively, if you want a platform expert who really understands the XPCOM interfaces being used here, I totally understand.
Whoops, over-zealous bug filing. :) I'll mark this fixed. If there are any changes you want to make, feel free to reopen.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
The Add-on SDK is no longer a Mozilla Labs experiment and has become a big enough project to warrant its own Bugzilla product, so the "Add-on SDK" product has been created for it, and I am moving its bugs to that product. To filter bugmail related to this change, filter on the word "looptid".
Component: Jetpack SDK → General
Product: Mozilla Labs → Add-on SDK
QA Contact: jetpack-sdk → general
Version: Trunk → unspecified
You need to log in before you can comment on or make changes to this bug.