GenericPrinter::put and its overrides should be MOZ_MUST_USE

NEW
Unassigned

Status

()

Core
JavaScript Engine
P3
normal
a year ago
2 months ago

People

(Reporter: Waldo, Unassigned)

Tracking

({triage-deferred})

Firefox Tracking Flags

(Not tracked)

Details

It returns int, but a bunch of places don't check for <0 meaning an error was reported.  Unfortunately, most of these uses are in the JIT, in APIs that don't have a way to propagate errors, so fixing this is a little sloggish.
I will precise, most of these uses are made for the JitSpew, using the LSprinter / Fprinter backend, which are only meant for debugging.
Keywords: triage-deferred
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.