From: Matthias Clasen <mclasen@redhat.com>
Date: Sat, 22 Dec 2007 18:15:57 +0000 (+0000)
Subject: Doc improvements
X-Git-Tag: GLIB_2_15_1~75
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1daa618e0529aed771a7ff5099463e75f721956f;p=platform%2Fupstream%2Fglib.git

Doc improvements

2007-12-22  Matthias Clasen  <mclasen@redhat.com>

        * gio/gappinfo.c: Doc improvements

        * gio/gdesktopappinfo.c (g_app_info_get_all): Return app infos,
        not ids.


svn path=/trunk/; revision=6188
---

diff --git a/gio/ChangeLog b/gio/ChangeLog
index c3ecc77..779fc1b 100644
--- a/gio/ChangeLog
+++ b/gio/ChangeLog
@@ -1,3 +1,10 @@
+2007-12-22  Matthias Clasen  <mclasen@redhat.com>
+
+	* gio/gappinfo.c: Doc improvements
+	
+	* gio/gdesktopappinfo.c (g_app_info_get_all): Return app infos,
+	not ids.
+
 2007-12-20  Matthias Clasen  <mclasen@redhat.com>
 
 	* === Released 2.15.0 ===
diff --git a/gio/gappinfo.c b/gio/gappinfo.c
index c2d4d5c..fb298f6 100644
--- a/gio/gappinfo.c
+++ b/gio/gappinfo.c
@@ -139,6 +139,9 @@ g_app_info_equal (GAppInfo *appinfo1,
  * platform dependent. For instance on Unix this is the
  * desktop file id from the xdg menu specification.
  *
+ * Note that the returned ID may be %NULL, depending on how
+ * the @appinfo has been constructed.
+ *
  * Returns: a string containing the application's ID.
  **/
 const char *
@@ -180,8 +183,7 @@ g_app_info_get_name (GAppInfo *appinfo)
  * Gets a human-readable description of an installed application.
  *
  * Returns: a string containing a description of the 
- * application @appinfo, or %NULL if none. The returned string should be not freed 
- * when no longer needed.
+ * application @appinfo, or %NULL if none. 
  **/
 const char *
 g_app_info_get_description (GAppInfo *appinfo)
diff --git a/gio/gdesktopappinfo.c b/gio/gdesktopappinfo.c
index 9f5d308..c89ae9b 100644
--- a/gio/gdesktopappinfo.c
+++ b/gio/gdesktopappinfo.c
@@ -920,10 +920,10 @@ static gboolean
 g_desktop_app_info_supports_uris (GAppInfo *appinfo)
 {
   GDesktopAppInfo *info = G_DESKTOP_APP_INFO (appinfo);
-  
-  return
-    (strstr (info->exec, "%u") != NULL) ||
-    (strstr (info->exec, "%U") != NULL);
+ 
+  return info->exec && 
+    ((strstr (info->exec, "%u") != NULL) ||
+     (strstr (info->exec, "%U") != NULL));
 }
 
 static gboolean
@@ -1581,7 +1581,8 @@ g_app_info_get_default_for_type (const char *content_type,
  * g_app_info_get_default_for_uri_scheme:
  * @uri_scheme: a string containing a URI scheme.
  *
- * Gets the default application for launching applications using this URI scheme.
+ * Gets the default application for launching applications 
+ * using this URI scheme.
  *
  * TODO: This is currently unimplemented.
  * 
@@ -1625,11 +1626,12 @@ get_apps_from_dir (GHashTable *apps,
 
 		  /* Don't return apps that don't take arguments */
 		  if (appinfo &&
-		      g_desktop_app_info_get_is_hidden (appinfo) &&
-		      strstr (appinfo->exec,"%U") == NULL &&
-		      strstr (appinfo->exec,"%u") == NULL &&
-		      strstr (appinfo->exec,"%f") == NULL &&
-		      strstr (appinfo->exec,"%F") == NULL)
+		      (g_desktop_app_info_get_is_hidden (appinfo) ||
+		       (appinfo->exec && 
+			strstr (appinfo->exec,"%U") == NULL &&
+			strstr (appinfo->exec,"%u") == NULL &&
+			strstr (appinfo->exec,"%f") == NULL &&
+			strstr (appinfo->exec,"%F") == NULL)))
 		    {
 		      g_object_unref (appinfo);
 		      appinfo = NULL;
@@ -1679,7 +1681,7 @@ g_app_info_get_all (void)
   const char * const *dirs;
   GHashTable *apps;
   GHashTableIter iter;
-  gpointer key;
+  gpointer value;
   int i;
   GList *infos;
 
@@ -1695,8 +1697,8 @@ g_app_info_get_all (void)
 
   infos = NULL;
   g_hash_table_iter_init (&iter, apps);
-  while (g_hash_table_iter_next (&iter, &key, NULL))
-    infos = g_list_prepend (infos, key);
+  while (g_hash_table_iter_next (&iter, NULL, &value))
+    infos = g_list_prepend (infos, value);
 
   g_hash_table_destroy (apps);
 
@@ -2142,13 +2144,13 @@ append_desktop_entry (GList      *list,
  * get_all_desktop_entries_for_mime_type:
  * @mime_type: a mime type.
  *
- * Returns all the desktop filenames for @mime_type. The desktop files
+ * Returns all the desktop ids for @mime_type. The desktop files
  * are listed in an order so that default applications are listed before
  * non-default ones, and handlers for inherited mimetypes are listed
  * after the base ones.
  *
- * Return value: a #GList containing the desktop filenames containing the
- * @mime_type.
+ * Return value: a #GList containing the desktop ids which claim
+ *    to handle @mime_type.
  */
 static GList *
 get_all_desktop_entries_for_mime_type (const char *base_mime_type)