<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [GStreamer][MSE] Some GStreamer log messages are generated with the 'default' category"
href="https://bugs.webkit.org/show_bug.cgi?id=168015#c6">Comment # 6</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [GStreamer][MSE] Some GStreamer log messages are generated with the 'default' category"
href="https://bugs.webkit.org/show_bug.cgi?id=168015">bug 168015</a>
from <span class="vcard"><a class="email" href="mailto:eocanha@igalia.com" title="Enrique Ocaña <eocanha@igalia.com>"> <span class="fn">Enrique Ocaña</span></a>
</span></b>
<pre>Comment on <span class=""><a href="attachment.cgi?id=300961&action=diff" name="attach_300961" title="Patch">attachment 300961</a> <a href="attachment.cgi?id=300961&action=edit" title="Patch">[details]</a></span>
Patch
View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=300961&action=review">https://bugs.webkit.org/attachment.cgi?id=300961&action=review</a>
<span class="quote">>> Source/WebCore/platform/graphics/gstreamer/mse/AppendPipeline.cpp:45
>> +#define GST_CAT_DEFAULT webkit_appendpipeline_debug
>
> Please use webkit_mse_debug, that is defined in MediaPlayerPrivateGStreamerMSE.cpp. For this you will need to make the declaration in MediaPlayerPrivateGStreamerMSE.cpp as _EXTERN as it is done for GStreamer and GStreamerBase.</span >
In any case, shouldn't the define GST_CAT_DEFAULT be still needed? I've explored the GST_TRACE() macro expansion and it expands to this code (in my particular case):
do{ \
if ((GST_LEVEL_TRACE <= GST_LEVEL_COUNT && GST_LEVEL_TRACE <= _gst_debug_min)) { \
gst_debug_log ((GST_CAT_DEFAULT), (GST_LEVEL_TRACE), "/home/enrique/WebKit/Source/WebCore/platform/graphics/gstreamer/mse/MediaSourceClientGStreamerMSE.cpp", ((const char*) (__FUNCTION__)), 65, \
(GObject *) (0), "Adding SourceBuffer to AppendPipeline: this=%p sourceBuffer=%p appendPipeline=%p", this, sourceBufferPrivate.get(), appendPipeline.get()); \
} \
}while (0)
This seems to imply that some value is needed for GST_CAT_DEFAULT. Since that define is inside MediaPlayerPrivateGStreamerMSE.cpp and not in the header (because it shouldn't be in the header in order to avoid clashing with local definitions when the header in included from other cpp files), it would remain undefined in the case of AppendPipeline, PlaybackPipeline and MediaSourceClientGStreamer. Therefore, I think we need to define it here by ourselves with the same value used in MediaPlayerPrivateGStreamerMSE.cpp:
#define GST_CAT_DEFAULT webkit_mse_debug</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>