diff options
| author | Stefan Sauer <ensonic@users.sf.net> | 2016-12-15 15:37:45 +0100 |
|---|---|---|
| committer | Stefan Sauer <ensonic@users.sf.net> | 2016-12-16 15:55:21 +0100 |
| commit | 1bfd04a5fee09a647a380d17bfe924f3860ad04c (patch) | |
| tree | cf4784815ce38eef7ebe1fe2f9f8bac3d59a943e /plugins | |
| parent | 564dc742a445aebc9394ef83aab7603536ef8818 (diff) | |
tracers/stats: log optional fields instead of GST_CLOCK_TIME_NONE
Simplify the traces and avoid trace analyzer to know that ((1<<64) - 1) means
we had no value.
Diffstat (limited to 'plugins')
| -rw-r--r-- | plugins/tracers/gststats.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/plugins/tracers/gststats.c b/plugins/tracers/gststats.c index 8b7705b5e..1a8f892a4 100644 --- a/plugins/tracers/gststats.c +++ b/plugins/tracers/gststats.c @@ -251,12 +251,15 @@ do_buffer_stats (GstStatsTracer * self, GstPad * this_pad, GstElementStats *this_elem_stats = get_element_stats (self, this_elem); GstElement *that_elem = get_real_pad_parent (that_pad); GstElementStats *that_elem_stats = get_element_stats (self, that_elem); + GstClockTime pts = GST_BUFFER_PTS (buf); + GstClockTime dts = GST_BUFFER_DTS (buf); + GstClockTime dur = GST_BUFFER_DURATION (buf); gst_tracer_record_log (tr_buffer, (guint64) (guintptr) g_thread_self (), elapsed, this_pad_stats->index, this_elem_stats->index, that_pad_stats->index, that_elem_stats->index, gst_buffer_get_size (buf), - GST_BUFFER_PTS (buf), GST_BUFFER_DTS (buf), GST_BUFFER_DURATION (buf), - GST_BUFFER_FLAGS (buf)); + GST_CLOCK_TIME_IS_VALID (pts), pts, GST_CLOCK_TIME_IS_VALID (dts), dts, + GST_CLOCK_TIME_IS_VALID (dur), dur, GST_BUFFER_FLAGS (buf)); } static void @@ -556,18 +559,21 @@ gst_stats_tracer_class_init (GstStatsTracerClass * klass) "buffer-pts", GST_TYPE_STRUCTURE, gst_structure_new ("value", "type", G_TYPE_GTYPE, G_TYPE_UINT64, "description", G_TYPE_STRING, "presentation timestamp of the buffer in ns", + "flags", GST_TYPE_TRACER_VALUE_FLAGS, GST_TRACER_VALUE_FLAGS_OPTIONAL, "min", G_TYPE_UINT64, G_GUINT64_CONSTANT (0), "max", G_TYPE_UINT64, G_MAXUINT64, NULL), "buffer-dts", GST_TYPE_STRUCTURE, gst_structure_new ("value", "type", G_TYPE_GTYPE, G_TYPE_UINT64, "description", G_TYPE_STRING, "decoding timestamp of the buffer in ns", + "flags", GST_TYPE_TRACER_VALUE_FLAGS, GST_TRACER_VALUE_FLAGS_OPTIONAL, "min", G_TYPE_UINT64, G_GUINT64_CONSTANT (0), "max", G_TYPE_UINT64, G_MAXUINT64, NULL), "buffer-duration", GST_TYPE_STRUCTURE, gst_structure_new ("value", "type", G_TYPE_GTYPE, G_TYPE_UINT64, "description", G_TYPE_STRING, "duration of the buffer in ns", + "flags", GST_TYPE_TRACER_VALUE_FLAGS, GST_TRACER_VALUE_FLAGS_OPTIONAL, "min", G_TYPE_UINT64, G_GUINT64_CONSTANT (0), "max", G_TYPE_UINT64, G_MAXUINT64, NULL), |