LarsGit223 commented on this pull request.
>
ctype = g_content_type_guess(path, NULL, 0, NULL);
icon = g_content_type_get_icon(ctype);
g_free(ctype);
if (icon != NULL)
{
- gtk_icon_size_lookup(GTK_ICON_SIZE_MENU, &width, NULL);
- info =
gtk_icon_theme_lookup_by_gicon(gtk_icon_theme_get_default(), icon, width,
GTK_ICON_LOOKUP_USE_BUILTIN);
- g_object_unref(icon);
+ info =
gtk_icon_theme_lookup_by_gicon(gtk_icon_theme_get_default(), icon, 16, 0);
Ok, then we leave it as-is. I didn't doubt that it works I just wondered if
there is a coding style for stuff like this which says 'use constants or
literals.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/927#discussion_r334326275
