[gstreamer-bugs] [Bug 380495] ClockID objects are useless

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=380495

  GStreamer | gst-python | Ver: HEAD CVS





------- Comment #9 from René Stadler  2006-12-16 15:25 UTC -------
That also occured to me afterwards, but like I said, the overrides are needed
anyways since we have to verify the time arguments and raise ValueErrors if
they are invalid.

In the long run, it makes lots of sense to add an argument attribute (or what
it's called) valid-clock-time, similar to the existing null-ok etc.  This would
allow us to define the methods like this:

(define-method new_periodic_id
  (of-object "GstClock")
  (c-name "gst_clock_new_periodic_id")
  (return-type "GstClockID")
  (parameters
    '("GstClockTime" "start_time" (valid-clock-time))
    '("GstClockTime" "interval" (valid-clock-time not-null)
  )
)

But this can be done as separate work, as it is relevant to lots of functions. 
It's a nice way to get rid of some overrides and add better checking to the
functions that miss it.


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email