2.34.0
[platform/upstream/at-spi2-core.git] / atspi / atspi-hyperlink.c
index fac4624..c01f547 100644 (file)
@@ -4,21 +4,22 @@
  *
  * Copyright 2001, 2002 Sun Microsystems Inc.,
  * Copyright 2001, 2002 Ximian, Inc.
+ * Copyright 2010, 2011 Novell, Inc.
  *
  * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Library General Public
+ * modify it under the terms of the GNU Lesser General Public
  * License as published by the Free Software Foundation; either
- * version 2 of the License, or (at your option) any later version.
+ * version 2.1 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * Library General Public License for more details.
+ * Lesser General Public License for more details.
  *
- * You should have received a copy of the GNU Library General Public
+ * You should have received a copy of the GNU Lesser General Public
  * License along with this library; if not, write to the
- * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
- * Boston, MA 02111-1307, USA.
+ * Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
+ * Boston, MA 02110-1301, USA.
  */
 
 #include "atspi-private.h"
@@ -36,13 +37,11 @@ atspi_hyperlink_class_init (AtspiHyperlinkClass *klass)
 }
 
 AtspiHyperlink *
-atspi_hyperlink_new (AtspiApplication *app, const gchar *path)
+_atspi_hyperlink_new (AtspiApplication *app, const gchar *path)
 {
   AtspiHyperlink *hyperlink;
   
   hyperlink = g_object_new (ATSPI_TYPE_HYPERLINK, NULL);
-  g_return_val_if_fail (hyperlink != NULL, NULL);
-
   hyperlink->parent.app = g_object_ref (app);
   hyperlink->parent.path = g_strdup (path);
 
@@ -53,18 +52,19 @@ atspi_hyperlink_new (AtspiApplication *app, const gchar *path)
  * atspi_hyperlink_get_n_anchors:
  * @obj: a pointer to the #AtspiHyperlink object on which to operate.
  *
- * Get the total number of anchors which an #AtspiHyperlink implementor has.
- *       Though typical hyperlinks have only one anchor, client-side image maps and
- *       other hypertext objects may potentially activate or refer to multiple
- *       URIs.  For each anchor there is a corresponding URI and object.
- * see atspi_hyperlink_get_uri() and atspi_hyperlink_get_object().
+ * Gets the total number of anchors which an #AtspiHyperlink implementor has.
+ * Though typical hyperlinks have only one anchor, client-side image maps and
+ * other hypertext objects may potentially activate or refer to multiple
+ * URIs.  For each anchor there is a corresponding URI and object.
+ *
+ * see: #atspi_hyperlink_get_uri and #atspi_hyperlink_get_object.
  *
  * Returns: a #gint indicating the number of anchors in this hyperlink.
  **/
 gint
 atspi_hyperlink_get_n_anchors (AtspiHyperlink *obj, GError **error)
 {
-  dbus_int32_t retval;
+  dbus_int32_t retval = -1;
 
   g_return_val_if_fail (obj != NULL, -1);
 
@@ -78,7 +78,7 @@ atspi_hyperlink_get_n_anchors (AtspiHyperlink *obj, GError **error)
  * @obj: a pointer to the #AtspiHyperlink implementor on which to operate.
  * @i: a (zero-index) integer indicating which hyperlink anchor to query.
  *
- * Get the URI associated with a particular hyperlink anchor.  
+ * Gets the URI associated with a particular hyperlink anchor.
  *
  * Returns: a UTF-8 string giving the URI of the @ith hyperlink anchor.
  **/
@@ -101,9 +101,10 @@ atspi_hyperlink_get_uri (AtspiHyperlink *obj, int i, GError **error)
 /**
  * atspi_hyperlink_get_object:
  * @obj: a pointer to the #AtspiHyperlink implementor on which to operate.
- * @i: a (zero-index) long integer indicating which hyperlink anchor to query.
+ * @i: a (zero-index) #gint indicating which hyperlink anchor to query.
  *
- * Get the object associated with a particular hyperlink anchor, as an #Accessible. 
+ * Gets the object associated with a particular hyperlink anchor, as an
+ * #AtspiAccessible.
  *
  * Returns: (transfer full): an #AtspiAccessible that represents the object
  *        associated with the @ith anchor of the specified #AtspiHyperlink.
@@ -126,19 +127,19 @@ atspi_hyperlink_get_object (AtspiHyperlink *obj, gint i, GError **error)
  * @obj: a pointer to the #AtspiHyperlink implementor on which to operate.
  *
  *
- * Get the starting and ending character offsets of the text range associated with
- *       a #AtspiHyperlink, in its originating #AtspiHypertext.
+ * Gets the starting and ending character offsets of the text range
+ * associated with an #AtspiHyperlink, in its originating #AtspiHypertext.
  **/
 AtspiRange *
 atspi_hyperlink_get_index_range (AtspiHyperlink *obj, GError **error)
 {
-  dbus_int32_t d_start_offset, d_end_offset;
+  dbus_int32_t d_start_offset = -1;
+  dbus_int32_t d_end_offset = -1;
   AtspiRange *ret = g_new (AtspiRange, 1);
 
-  if (ret)
-    ret->start_offset = ret->end_offset = -1;
+  ret->start_offset = ret->end_offset = -1;
 
-  if (!obj || !ret)
+  if (!obj)
     return ret;
 
   _atspi_dbus_call (obj, atspi_interface_hyperlink, "GetIndexRange", error, "=>ii", &d_start_offset, &d_end_offset);
@@ -153,8 +154,8 @@ atspi_hyperlink_get_index_range (AtspiHyperlink *obj, GError **error)
  * @obj: a pointer to the #AtspiHyperlink implementor on which to operate.
  *
  *
- * Get the starting character offset of the text range associated with
- *       a #AtspiHyperlink, in its originating #AtspiHypertext.
+ * Gets the starting character offset of the text range associated with
+ *       an #AtspiHyperlink, in its originating #AtspiHypertext.
  **/
 gint
 atspi_hyperlink_get_start_index (AtspiHyperlink *obj, GError **error)
@@ -174,8 +175,8 @@ atspi_hyperlink_get_start_index (AtspiHyperlink *obj, GError **error)
  * @obj: a pointer to the #AtspiHyperlink implementor on which to operate.
  *
  *
- * Get the ending character offset of the text range associated with
- *       a #AtspiHyperlink, in its originating #AtspiHypertext.
+ * Gets the ending character offset of the text range associated with
+ *       an #AtspiHyperlink, in its originating #AtspiHypertext.
  **/
 gint
 atspi_hyperlink_get_end_index (AtspiHyperlink *obj, GError **error)
@@ -196,16 +197,16 @@ atspi_hyperlink_get_end_index (AtspiHyperlink *obj, GError **error)
  * atspi_hyperlink_is_valid:
  * @obj: a pointer to the #AtspiHyperlink on which to operate.
  *
- * Tell whether an #AtspiHyperlink object is still valid with respect to its
+ * Tells whether an #AtspiHyperlink object is still valid with respect to its
  *          originating hypertext object.
  *
- * Returns: #TRUE of the specified #AtspiHyperlink is still valid with respect
+ * Returns: #TRUE if the specified #AtspiHyperlink is still valid with respect
  *          to its originating #AtspiHypertext object, #FALSE otherwise.
  **/
 gboolean
 atspi_hyperlink_is_valid (AtspiHyperlink *obj, GError **error)
 {
-  dbus_bool_t retval;
+  dbus_bool_t retval = FALSE;
 
   g_return_val_if_fail (obj != NULL, FALSE);