[Bug 680237] QGst::ClockTime is inconvenient to use

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=680237
  GStreamer | qt-gstreamer | git

George Kiagiadakis <kiagiadakis.george> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kiagiadakis.george at gmail.co
                   |                            |m

--- Comment #4 from George Kiagiadakis <kiagiadakis.george at gmail.com> 2012-08-02 08:43:17 UTC ---
(In reply to comment #2)
> (In reply to comment #1)
> > Just on a side note, GStreamer defines GST_SECOND, GST_MSECOND etc. so you can
> > write 5 * GST_SECOND or 5 * GST_MSECOND to get the right amount of nanoseconds
> > (but explicit constructors are nicer of course).
> 
> QtGStreamer explicitly doesn't include the GStreamer headers to avoid bad
> interactions between Qt convenience macros (slots, signals, foreach, forever,
> ...) and GLib related headers.

Fwiw, the reason that QtGStreamer does not include GStreamer headers in its
public headers is not that. The reason is that I don't want applications to be
polluted with too many includes that are not necessarily required. This way,
you don't have to install the gstreamer, glib and libxml development packages
in order to use QtGStreamer and you also don't need to force applications to
link to gstreamer and glib. This makes compilation and linking faster and
serves as an abstraction layer in the (highly unlikely) case gstreamer and/or
glib break ABI without API at some point (only QtGStreamer would require
recompilation).

I admit though that having some macros from GStreamer would be convenient.

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