[Bug 756806] message: Add extra information fields to error/warning/info messages

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=756806

--- Comment #13 from Vincent Penquerc'h <vincent.penquerch at collabora.co.uk> ---
(In reply to Tim-Philipp Müller from comment #12)

[misc annotation bugs]

Fixed (though not posting because of the following)

> > #define GST_ELEMENT_ERROR_WITH_DETAILS(el, domain, code, text, debug, args...)
> 
> Variadic macros are a bit tricky, there are multiple flavours and we need a
> backup for when not available, see gstinfo.h

That's going to be tricky with the ## usage. One'd need a macro to paste those,
unless making new identifiers that'd trample all over the global namespace.

> - I wonder if the detail fields should be set as a separate structure or
> actually directly on the existing GstMessage structure?

There's a larger risk of namespace collision if using the same structure.

No real opinion on the copy-or-not choice.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.