[timeline] Check the fps property range in the ctor
authorEmmanuele Bassi <ebassi@linux.intel.com>
Tue, 27 Jan 2009 10:35:50 +0000 (10:35 +0000)
committerEmmanuele Bassi <ebassi@linux.intel.com>
Tue, 27 Jan 2009 10:35:50 +0000 (10:35 +0000)
The clutter_timeline_new() constructor is not checking the full
range of the passed :fps property. The ParamSpec of the property
and the setter method perform this check, so the ctor should as
well.

clutter/clutter-timeline.c

index 739724c..47bf9ca 100644 (file)
@@ -1258,7 +1258,7 @@ clutter_timeline_new (guint n_frames,
                      guint fps)
 {
   g_return_val_if_fail (n_frames > 0, NULL);
-  g_return_val_if_fail (fps > 0, NULL);
+  g_return_val_if_fail (fps > 0 && fps <= 1000, NULL);
 
   return g_object_new (CLUTTER_TYPE_TIMELINE,
                       "fps", fps,