[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