[Bug 778702] videotimecode: Init from GDateTime

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Thu Feb 16 17:32:07 UTC 2017


https://bugzilla.gnome.org/show_bug.cgi?id=778702

--- Comment #4 from Vivia Nikolaidou <vivia at ahiru.eu> ---
Review of attachment 345950:
 --> (https://bugzilla.gnome.org/review?bug=778702&attachment=345950)

Looks good generally, but don't forget to add some unit tests as well!

::: gst-libs/gst/video/gstvideotimecode.c
@@ +226,3 @@
+      g_date_time_get_second (dt), frames, field_count);
+
+  gst_video_time_code_add_frames (tc, frames);

Why do you init using the frames and then re-add the frames?

You should also add a check in case the timecode is invalid, e.g. 01:02:03;00

-- 
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