Closed Bug 2039586 Opened 6 months ago Closed 5 months ago

Simplify the xpcom_async crate

Categories

(MailNews Core :: Networking: Exchange, task)

task
Points:
3

Tracking

(Not tracked)

RESOLVED FIXED
153 Branch

People

(Reporter: babolivier, Assigned: babolivier)

References

(Blocks 1 open bug)

Details

Attachments

(2 external files)

The xpcom_async crate was initially built with the idea that XpComFuture could be expanded to other XPCOM types than nsIChannel. This isn't exactly trivial, and we don't really have a need for that, and the crate's timer handling already deviate from that design. It should be (lightly) refactored to better represent what the crate actually is: a grab bag of utilities to bridge async XPCOM with async Rust.

Points: --- → 3
Status: NEW → ASSIGNED

The main reason I've made this change is because I was told some time ago the
name of this crate was confusing when coming to the outside, and meant the crate
could be confused with an async runtime.

Target Milestone: --- → 153 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/6e3518cecc3d
Part 1: Replace generic XpComFuture with AsyncChannelOpener. r=edicharry
https://hg.mozilla.org/comm-central/rev/b8acdfc311e1
Part 2: Rename xpcom_async into xpcom_async_glue. r=edicharry

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

Attachment

General

Created:
Updated:
Size: