From d038c85495317daee3d036a48d519acc9ceb2727 Mon Sep 17 00:00:00 2001 From: Ted Gould Date: Tue, 22 Apr 2014 10:17:03 -0500 Subject: Unroll one more level of if statement --- src/accounts-service-user.vala | 21 ++++++++++----------- 1 file changed, 10 insertions(+), 11 deletions(-) (limited to 'src/accounts-service-user.vala') diff --git a/src/accounts-service-user.vala b/src/accounts-service-user.vala index 194beb0..04c38cc 100644 --- a/src/accounts-service-user.vala +++ b/src/accounts-service-user.vala @@ -58,20 +58,19 @@ public class AccountsServiceUser : Object { } else { this.proxy.timestamp = GLib.get_monotonic_time(); this.proxy.player_name = this._player.name; - if (this._player.icon == null) { - var icon = new ThemedIcon.with_default_fallbacks ("application-default-icon"); - this.proxy.player_icon = icon.serialize(); - } else { - var serialization = this._player.icon.serialize(); - if (serialization == null) { - var icon = new ThemedIcon.with_default_fallbacks ("application-default-icon"); - serialization = icon.serialize(); - } - - this.proxy.player_icon = serialization; + /* Serialize the icon if it exits, if it doesn't or errors then + we need to use the application default icon */ + GLib.Variant? icon_serialization = null; + if (this._player.icon != null) + icon_serialization = this._player.icon.serialize(); + if (icon_serialization == null) { + var icon = new ThemedIcon.with_default_fallbacks ("application-default-icon"); + icon_serialization = icon.serialize(); } + this.proxy.player_icon = icon_serialization; + /* Set state of the player */ this.proxy.running = this._player.is_running; this.proxy.state = this._player.state; -- cgit v1.2.3