Closed Bug 1697320 Opened 4 years ago Closed 4 years ago

webrender::render_task_graph::dump_render_tasks_as_svg should not be marked #[allow(dead_code)]

Categories

(Core :: Graphics: WebRender, defect, P4)

defect

Tracking

()

RESOLVED FIXED
88 Branch
Tracking Status
firefox88 --- fixed

People

(Reporter: jimb, Assigned: jimb)

Details

Attachments

(1 file)

The reason Rust didn't warn us that RenderPass::tasks was unused (bug 1695908) is that dump_render_tasks_as_svg contains a use of that field, and is marked as #[allow(dead-code)]. If it were instead marked #[cfg(feature = "capture")], then Rust could have warned us about the dead field in non-capture builds.

Assignee: nobody → jimb
Status: NEW → ASSIGNED
Pushed by jblandy@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/537e54c210e7 Mark webrender::render_task_graph::dump_render_tasks_as_svg as needed only when 'capture' feature is enabled r=nical
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: