Open Bug 1992119 Opened 1 month ago

Consider introducing a TestGetNumErrors trait to put `test_get_num_recorded_errors` in

Categories

(Data Platform and Tools :: Glean: SDK, defect, P3)

Product:
Data Platform and Tools
Component:
Glean: SDK
Type:
defect

Tracking

(Not tracked)

People

(Reporter: janerik, Unassigned)

References

Details

(Whiteboard: [telemetry:glean-rs:m?])

In bug 1932617 we move test_get_num_recorded_errors out of the trait Event.
This is required in order to take dyn objects in some of our generated code to reduce code size.

Moving it out is not a huge issue. We're the only implementors right now and we added those traits so that we ourselves don't forget to implement all the right functions. We do that for events now regardless on whether we're forced to or not.
We can consider doing this for all traits though.

You need to log in before you can comment on or make changes to this bug.