luc14n0/glib
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/glib.git synced 2025-01-12 07:26:15 +01:00
Code Issues Packages Projects Releases Wiki Activity

gfile: Use async API to get the default GAppInfo for type

Browse Source 
In both callbacks of g_file_query_default_handler_async() we were
actually using I/O blocking APIs making it not fully async.

Now that such API is provided, we can use it.
This commit is contained in:
Marco Trevisan (Treviño) 2022-06-02 04:33:00 +02:00
parent 44dbd43170
commit c87602451c
1 changed files with 45 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

58
gio/gfile.c
View File

@ -7523,6 +7523,36 @@ g_file_query_default_handler (GFile *file,
return NULL;
}
static void
query_default_handler_query_app_info_for_type_cb (GObject *object,
GAsyncResult *result,
gpointer user_data)
{
GTask *task = G_TASK (user_data);
GAppInfo *appinfo;
GError *error = NULL;
appinfo = g_app_info_get_default_for_type_finish (result, &error);
if (appinfo != NULL)
{
g_task_return_pointer (task, g_steal_pointer (&appinfo), g_object_unref);
}
else if (g_error_matches (error, G_IO_ERROR, G_IO_ERROR_NOT_FOUND))
{
g_task_return_new_error (task,
G_IO_ERROR, G_IO_ERROR_NOT_SUPPORTED,
"%s", error->message);
}
else
{
g_task_return_error (task, g_steal_pointer (&error));
}
g_clear_error (&error);
g_object_unref (task);
}
static void
query_default_handler_query_info_cb (GObject *object,
GAsyncResult *result,
@ -7533,7 +7563,6 @@ query_default_handler_query_info_cb (GObject *object,
GError *error = NULL;
GFileInfo *info;
const char *content_type;
GAppInfo *appinfo = NULL;
info = g_file_query_info_finish (file, result, &error);
if (info == NULL)
@ -7548,27 +7577,30 @@ query_default_handler_query_info_cb (GObject *object,
content_type = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_STANDARD_FAST_CONTENT_TYPE);
if (content_type)
{
GCancellable *cancellable = g_task_get_cancellable (task);
char *path;
/* Don't use is_native(), as we want to support fuse paths if available */
path = g_file_get_path (file);
/* FIXME: The following still uses blocking calls. */
appinfo = g_app_info_get_default_for_type (content_type,
path == NULL);
g_app_info_get_default_for_type_async (content_type,
path == NULL,
cancellable,
query_default_handler_query_app_info_for_type_cb,
g_steal_pointer (&task));
g_free (path);
}
else
{
g_task_return_new_error (task,
G_IO_ERROR,
G_IO_ERROR_NOT_SUPPORTED,
_("No application is registered as handling this file"));
}
g_object_unref (info);
if (appinfo != NULL)
g_task_return_pointer (task, g_steal_pointer (&appinfo), g_object_unref);
else
g_task_return_new_error (task,
G_IO_ERROR,
G_IO_ERROR_NOT_SUPPORTED,
_("No application is registered as handling this file"));
g_object_unref (task);
g_clear_object (&task);
}
/**