Check given parameters to public API

diff --git a/champlain/champlainmarker.c b/champlain/champlainmarker.c
index 1ff15a597d0a7bb9e35cc64798b0332a2b409ce5..8c9593031ad7dc6d9b682b399b3095d14755226f 100644 (file)
--- a/champlain/champlainmarker.c
+++ b/champlain/champlainmarker.c
@@ -190,6 +190,8 @@ champlain_marker_new ()
 void
 champlain_marker_set_position (ChamplainMarker *champlainMarker, gdouble latitude, gdouble longitude)
 {
+  g_return_if_fail(CHAMPLAIN_IS_MARKER(champlainMarker));
+
   ChamplainMarkerPrivate *priv = CHAMPLAIN_MARKER_GET_PRIVATE (champlainMarker);
 
   priv->lon = longitude;

diff --git a/champlain/champlainview.c b/champlain/champlainview.c
index 874660d61181e5bea134fe62b5900d2f784080b8..5be05a6c0e9d2400cd5b244e7fe4881f213482fa 100644 (file)
--- a/champlain/champlainview.c
+++ b/champlain/champlainview.c
@@ -578,8 +578,10 @@ viewport_x_changed_cb(GObject *gobject, GParamSpec *arg1, ChamplainView *view)
 }
 
 void
-champlain_view_set_size (ChamplainView *view, gint width, gint height)
+champlain_view_set_size (ChamplainView *view, guint width, guint height)
 {
+  g_return_if_fail(CHAMPLAIN_IS_VIEW(view));
+
   ChamplainViewPrivate *priv = CHAMPLAIN_VIEW_GET_PRIVATE (view);
 
   priv->viewport_size.width = width;
@@ -669,6 +671,8 @@ champlain_view_new (ChamplainViewMode mode)
 void
 champlain_view_center_on (ChamplainView *view, gdouble latitude, gdouble longitude)
 {
+  g_return_if_fail(CHAMPLAIN_IS_VIEW(view));
+
   ChamplainViewPrivate *priv = CHAMPLAIN_VIEW_GET_PRIVATE (view);
 
   if(!priv->map)
@@ -737,6 +741,8 @@ champlain_view_center_on (ChamplainView *view, gdouble latitude, gdouble longitu
 void
 champlain_view_zoom_in (ChamplainView *view)
 {
+  g_return_if_fail(CHAMPLAIN_IS_VIEW(view));
+
   ChamplainViewPrivate *priv = CHAMPLAIN_VIEW_GET_PRIVATE (view);
   ClutterActor *group = priv->map->current_level->group;
   gdouble lon = viewport_get_current_longitude(priv);
@@ -764,6 +770,8 @@ champlain_view_zoom_in (ChamplainView *view)
 void
 champlain_view_zoom_out (ChamplainView *view)
 {
+  g_return_if_fail(CHAMPLAIN_IS_VIEW(view));
+
   ChamplainViewPrivate *priv = CHAMPLAIN_VIEW_GET_PRIVATE (view);
   ClutterActor *group = priv->map->current_level->group;
   gdouble lon = viewport_get_current_longitude(priv);
@@ -792,6 +800,9 @@ champlain_view_zoom_out (ChamplainView *view)
 void
 champlain_view_add_layer (ChamplainView *view, ClutterActor *layer)
 {
+  g_return_if_fail(CHAMPLAIN_IS_VIEW(view));
+  g_return_if_fail(CLUTTER_IS_ACTOR(layer));
+
   ChamplainViewPrivate *priv = CHAMPLAIN_VIEW_GET_PRIVATE (view);
   clutter_container_add(CLUTTER_CONTAINER(priv->user_layers), layer, NULL);
   clutter_actor_raise_top(layer);

diff --git a/champlain/champlainview.h b/champlain/champlainview.h
index 603550556d8ffd9ac67a151ddf80690d0e4628d8..c98a8dd49656987556e78f43467cd3c2a96a81c0 100644 (file)
--- a/champlain/champlainview.h
+++ b/champlain/champlainview.h
@@ -89,6 +89,6 @@ void champlain_view_zoom_out (ChamplainView *champlainView);
 
 void champlain_view_add_layer (ChamplainView *champlainView, ClutterActor *layer);
 
-void champlain_view_set_size (ChamplainView *view, gint width, gint height);
+void champlain_view_set_size (ChamplainView *view, guint width, guint height);
 
 #endif