After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 766466 - kmssink: enhance logs
kmssink: enhance logs
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
unspecified
Other All
: Low enhancement
: 1.9.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-05-15 10:03 UTC by Víctor Manuel Jáquez Leal
Modified: 2016-10-31 14:23 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
kmssink: use trace level for buffer render log (929 bytes, patch)
2016-05-15 10:03 UTC, Víctor Manuel Jáquez Leal
committed Details | Review
kmssink: frame copy log in performance category (1.48 KB, patch)
2016-05-15 10:04 UTC, Víctor Manuel Jáquez Leal
committed Details | Review

Description Víctor Manuel Jáquez Leal 2016-05-15 10:03:50 UTC
Look the patch logs
Comment 1 Víctor Manuel Jáquez Leal 2016-05-15 10:03:55 UTC
Created attachment 327923 [details] [review]
kmssink: use trace level for buffer render log

Instead of using debug level for logging each buffer rendered, use trace
level.
Comment 2 Víctor Manuel Jáquez Leal 2016-05-15 10:04:02 UTC
Created attachment 327924 [details] [review]
kmssink: frame copy log in performance category

Log the message when the frame is going to be copy (worse case) under the
performance log category.
Comment 3 Tim-Philipp Müller 2016-05-20 20:28:00 UTC
Any reason why you didn't just push this? I think you should feel free to push stuff like that, even more so seeing that you wrote the plugin.
Comment 4 Víctor Manuel Jáquez Leal 2016-05-21 08:58:22 UTC
(In reply to Tim-Philipp Müller from comment #3)
> Any reason why you didn't just push this? I think you should feel free to
> push stuff like that, even more so seeing that you wrote the plugin.

Because I haven't tested it yet :)

But I'll do along the weekend, with another patch for an issue that Nicolas found.
Comment 5 Víctor Manuel Jáquez Leal 2016-05-23 20:09:26 UTC
Attachment 327923 [details] pushed as e87772d - kmssink: use trace level for buffer render log
Attachment 327924 [details] pushed as 3b87265 - kmssink: frame copy log in performance category