[gnome-games/simplify-thumbnail: 24/28] game-thumbnail: Add IconInfo null checks
- From: Abhinav Singh <abhinavsingh src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-games/simplify-thumbnail: 24/28] game-thumbnail: Add IconInfo null checks
- Date: Mon, 8 Oct 2018 22:41:59 +0000 (UTC)
commit 3b4234184c72a7580939b1e5bc2fd38ae5614970
Author: theawless <theawless gmail com>
Date: Tue Oct 2 16:22:25 2018 +0530
game-thumbnail: Add IconInfo null checks
Also improve the related warning messages.
src/ui/game-thumbnail.vala | 24 +++++++++++++++++-------
1 file changed, 17 insertions(+), 7 deletions(-)
---
diff --git a/src/ui/game-thumbnail.vala b/src/ui/game-thumbnail.vala
index 1a617452..91ae76a7 100644
--- a/src/ui/game-thumbnail.vala
+++ b/src/ui/game-thumbnail.vala
@@ -152,11 +152,16 @@ private class Games.GameThumbnail: Gtk.DrawingArea {
var theme = Gtk.IconTheme.get_default ();
var size = int.min (context.width, context.height) * scale;
+ var icon_info = theme.lookup_icon (icon_name, (int) size, Gtk.IconLookupFlags.FORCE_SIZE);
+
+ if (icon_info == null) {
+ warning ("Couldn't find the emblem");
+ return;
+ }
try {
- var icon_info = theme.lookup_icon (icon_name, (int) size,
Gtk.IconLookupFlags.FORCE_SIZE);
emblem = icon_info.load_symbolic (color);
- } catch (Error error) {
- warning (@"Unable to get icon “$icon_name”: $(error.message)");
+ } catch (Error e) {
+ warning (@"Couldn’t load the emblem: $(e.message)");
return;
}
@@ -179,14 +184,15 @@ private class Games.GameThumbnail: Gtk.DrawingArea {
var size = int.min (context.width, context.height) * scale;
var icon_info = theme.lookup_by_gicon (icon, (int) size, lookup_flags);
- if (icon_info == null)
+ if (icon_info == null) {
+ warning ("Couldn't find the icon");
return null;
-
+ }
try {
return icon_info.load_icon ();
}
catch (Error e) {
- warning (@"Couldn’t load the icon: $(e.message)\n");
+ warning (@"Couldn’t load the icon: $(e.message)");
return null;
}
}
@@ -221,12 +227,16 @@ private class Games.GameThumbnail: Gtk.DrawingArea {
var lookup_flags = Gtk.IconLookupFlags.FORCE_SIZE | Gtk.IconLookupFlags.FORCE_REGULAR;
var icon_info = theme.lookup_by_gicon (g_icon, (int) size, lookup_flags);
+ if (icon_info == null) {
+ warning ("Couldn't find the cover");
+ return null;
+ }
try {
cover_cache = icon_info.load_icon ();
save_cover_cache_to_disk (size);
}
catch (Error e) {
- warning (@"Couldn’t load the icon: $(e.message)\n");
+ warning (@"Couldn’t load the cover: $(e.message)");
}
return cover_cache;
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]