ges: Fix documentation and debug comments

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D1393

diff --git a/ges/ges-timeline-element.c b/ges/ges-timeline-element.c
index 9512cb6..0c4ca46 100644 (file)
--- a/ges/ges-timeline-element.c
+++ b/ges/ges-timeline-element.c
@@ -656,10 +656,10 @@ ges_timeline_element_get_timeline (GESTimelineElement * self)
  * @self: a #GESTimelineElement
  * @start: the position in #GstClockTime
  *
- * Set the position of the object in its containing layer
+ * Set the position of the object in its containing layer.
  *
- * Note that if the timeline snap-distance property of the timeline containing
- * @self is set, @self will properly snap to its neighboors.
+ * Note that if the snapping-distance property of the timeline containing
+ * @self is set, @self will properly snap to the edges around @start.
  */
 void
 ges_timeline_element_set_start (GESTimelineElement * self, GstClockTime start)
@@ -678,7 +678,7 @@ ges_timeline_element_set_start (GESTimelineElement * self, GstClockTime start)
   toplevel_container = ges_timeline_element_get_toplevel_parent (self);
 
   if (((gint64) (_START (toplevel_container) + start - _START (self))) < 0) {
-    GST_INFO_OBJECT (self, "Can not move the object as it would imply its"
+    GST_INFO_OBJECT (self, "Can not move the object as it would imply its "
         "container to have a negative start value");
 
     gst_object_unref (toplevel_container);

diff --git a/ges/ges-timeline.c b/ges/ges-timeline.c
index cd32a37..6079077 100644 (file)
--- a/ges/ges-timeline.c
+++ b/ges/ges-timeline.c
@@ -2016,7 +2016,7 @@ ges_timeline_move_object_simple (GESTimeline * timeline,
   GESContainer *toplevel;
 
   /* We only work with GESSource-s and we check that we are not already moving
-   * element ourself*/
+   * the specified element ourself */
   if (GES_IS_SOURCE (element) == FALSE ||
       g_list_find (timeline->priv->movecontext.moving_trackelements, element))
     return FALSE;
@@ -2029,7 +2029,7 @@ ges_timeline_move_object_simple (GESTimeline * timeline,
   top_end = _START (toplevel) + _DURATION (toplevel) + position_offset;
   cur = g_hash_table_lookup (timeline->priv->by_end, track_element);
 
-  GST_DEBUG_OBJECT (timeline, "Moving %" GST_PTR_FORMAT "to %"
+  GST_DEBUG_OBJECT (timeline, "Moving %" GST_PTR_FORMAT " to %"
       GST_TIME_FORMAT " (end %" GST_TIME_FORMAT ")", element,
       GST_TIME_ARGS (position), GST_TIME_ARGS (top_end));
 

diff --git a/ges/ges-track-element.c b/ges/ges-track-element.c
index 21f6ca4..cfd5a1f 100644 (file)
--- a/ges/ges-track-element.c
+++ b/ges/ges-track-element.c
@@ -1340,7 +1340,7 @@ ges_track_element_copy_bindings (GESTrackElement * element,
  *  happen in, %NULL means that the edition is done in all the
  *  #GESLayers contained in the current timeline.
  *      FIXME: This is not implemented yet.
- * @mode: The #GESEditMode in which the editition will happen.
+ * @mode: The #GESEditMode in which the edition will happen.
  * @edge: The #GESEdge the edit should happen on.
  * @position: The position at which to edit @object (in nanosecond)
  *
@@ -1360,7 +1360,7 @@ ges_track_element_edit (GESTrackElement * object,
   g_return_val_if_fail (GES_IS_TRACK_ELEMENT (object), FALSE);
 
   if (G_UNLIKELY (!track)) {
-    GST_WARNING_OBJECT (object, "Trying to edit in %d mode but not in"
+    GST_WARNING_OBJECT (object, "Trying to edit in %d mode but not in "
         "any Track yet.", mode);
     return FALSE;
   }
@@ -1368,8 +1368,8 @@ ges_track_element_edit (GESTrackElement * object,
   timeline = GES_TIMELINE (ges_track_get_timeline (track));
 
   if (G_UNLIKELY (!timeline)) {
-    GST_WARNING_OBJECT (object, "Trying to edit in %d mode but not in"
-        "track %p no in any timeline yet.", mode, track);
+    GST_WARNING_OBJECT (object, "Trying to edit in %d mode but "
+        "track %p is not in any timeline yet.", mode, track);
     return FALSE;
   }