Closed Bug 1677441 Opened 4 years ago Closed 4 years ago

Refactor `dispatcher::launch(.... with_glean(|..|...))` to a single function (RLB)

Categories

(Data Platform and Tools :: Glean: SDK, enhancement, P1)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Dexter, Assigned: chinyingli.alice, Mentored)

Details

(Whiteboard: [good next bug][lang=Rust])

Attachments

(1 file)

As Jan-Erik rightfully pointed out on github, code such as:

dispatcher::launch(move || crate::with_glean(|glean| metric.add(glean, amount)));

in the rust language bindings could be refactored to a single function like:

launch_with_glean(|glean| ...)

which would internally call the dispatcher and acquire the Glean instance.

This is a potentially good next bug.

Whiteboard: [telemetry:fog:m?] → [telemetry:glean-rs:m?]

ni?myself to fill in the steps to help

Mentor: jrediger
Flags: needinfo?(jrediger)
Whiteboard: [telemetry:glean-rs:m?] → [telemetry:glean-rs:m?][good next bug][lang=Rust]
Flags: needinfo?(jrediger)
Whiteboard: [telemetry:glean-rs:m?][good next bug][lang=Rust] → [telemetry:glean-rs:backlog][good next bug][lang=Rust]
Priority: P3 → P4
Assignee: nobody → chinyingli.alice
Priority: P4 → P1
Whiteboard: [telemetry:glean-rs:backlog][good next bug][lang=Rust] → [good next bug][lang=Rust]
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: