Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

pvr: only update channel IconPath if the comparison channel IconPath …

…is not empty (extra check).

This avoids persisting unchanged channels that have their icons set automatically in XBMC but do not have an icon set in the backend.
Speeds up initial channel loading time, since only *actually* changed channels are persisted (compared to the current behavior which is all channels with an icon + changed channels)
  • Loading branch information...
commit 0ef1b2ca4bfb4b4e0644f52472511ffe65d7a301 1 parent b6c393b
@nemphys authored
Showing with 1 addition and 1 deletion.
  1. +1 −1  xbmc/pvr/channels/PVRChannel.cpp
View
2  xbmc/pvr/channels/PVRChannel.cpp
@@ -212,7 +212,7 @@ bool CPVRChannel::UpdateFromClient(const CPVRChannel &channel)
CSingleLock lock(m_critSection);
if (m_strChannelName.IsEmpty())
SetChannelName(channel.ClientChannelName());
- if (m_strIconPath.IsEmpty()||(!m_strIconPath.Equals(channel.IconPath()) && !IsUserSetIcon()))
+ if (m_strIconPath.IsEmpty()||(!channel.IconPath().IsEmpty() && !m_strIconPath.Equals(channel.IconPath()) && !IsUserSetIcon()))
SetIconPath(channel.IconPath());
return m_bChanged;
Please sign in to comment.
Something went wrong with that request. Please try again.