[Bug 705189] New: debug-dot-dump: Add the presence/state of a GstTask
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Wed Jul 31 00:33:07 PDT 2013
https://bugzilla.gnome.org/show_bug.cgi?id=705189
GStreamer | gstreamer (core) | git
Summary: debug-dot-dump: Add the presence/state of a GstTask
Classification: Platform
Product: GStreamer
Version: git
OS/Version: Linux
Status: UNCONFIRMED
Severity: normal
Priority: Normal
Component: gstreamer (core)
AssignedTo: gstreamer-bugs at lists.freedesktop.org
ReportedBy: olivier.crete at ocrete.ca
QAContact: gstreamer-bugs at lists.freedesktop.org
GNOME version: ---
Created an attachment (id=250521)
View: https://bugzilla.gnome.org/attachment.cgi?id=250521
Review: https://bugzilla.gnome.org/review?bug=705189&attachment=250521
debugutils: Print if there is a task started from a pad
For those who don't know how each element is implemented, it's not obvious
which ones have their own threads and which ones don't. Attached patch adds a
[T] for started tasks and a [t] for paused tasks if the dot files request the
state to be printed.
Am I right to take the object lock of the pad? I noticed the rest of the dot
file logic never takes any lock. As I'm dereferencing a pointer, I'm afraid of
non-atomic pointer writes could cause a crash otherwise.
Any stylistic preferences? Should we make it into another flag to enable it?
Maybe change the color of the pad slightly ?
--
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.
More information about the gstreamer-bugs
mailing list