{"action":"created","issue":{"url":"https://api.github.com/repos/xbmc/xbmc/issues/12300","repository_url":"https://api.github.com/repos/xbmc/xbmc","labels_url":"https://api.github.com/repos/xbmc/xbmc/issues/12300/labels{/name}","comments_url":"https://api.github.com/repos/xbmc/xbmc/issues/12300/comments","events_url":"https://api.github.com/repos/xbmc/xbmc/issues/12300/events","html_url":"https://github.com/xbmc/xbmc/pull/12300","id":235820172,"number":12300,"title":"[GUI] Prevent parse step of VisibleConditions if expression is already…","user":{"login":"peak3d","id":22704999,"avatar_url":"https://avatars0.githubusercontent.com/u/22704999?v=3","gravatar_id":"","url":"https://api.github.com/users/peak3d","html_url":"https://github.com/peak3d","followers_url":"https://api.github.com/users/peak3d/followers","following_url":"https://api.github.com/users/peak3d/following{/other_user}","gists_url":"https://api.github.com/users/peak3d/gists{/gist_id}","starred_url":"https://api.github.com/users/peak3d/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peak3d/subscriptions","organizations_url":"https://api.github.com/users/peak3d/orgs","repos_url":"https://api.github.com/users/peak3d/repos","events_url":"https://api.github.com/users/peak3d/events{/privacy}","received_events_url":"https://api.github.com/users/peak3d/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2017-06-14T09:37:59Z","updated_at":"2017-06-14T18:47:49Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/xbmc/xbmc/pulls/12300","html_url":"https://github.com/xbmc/xbmc/pull/12300","diff_url":"https://github.com/xbmc/xbmc/pull/12300.diff","patch_url":"https://github.com/xbmc/xbmc/pull/12300.patch"},"body":"… in CInfoManager::m_bools / Reset queued animations if they get active / prevent refresh-update for invisible controls\r\n\r\n## Description\r\n- Currently InfoBools of Type InfoSimple and InfoExpression are evaluated on ctor time.\r\nBecause we have a unique list (CInfoManager::m_bools) we now do the parsing only if we insert a new expression.\r\n- For some animation conditions we don't reset the animation type after animation wich leads to dirty region wich never disappears. Fixed (but must be kept in mind that we changed something here)\r\n- Invisible CControlGroup parents currently evaluate CLabel changes and set region dirty. For SmartRedraw (PR 12213) this is contra-productive.\r\n- Currently CGUIDialogAddonInfo works with the shared_ptr of an CGUIListItem, wich leads to issues when terminating kodi (one ListItem -> two different parents). Instead working with the shared_ptr we copy the content.\r\n\r\n## Motivation and Context\r\nOptimization general / smartredraw.\r\n\r\n## How Has This Been Tested?\r\nWindows 10 Estuary / Estuary Mod V2\r\n"},"comment":{"url":"https://api.github.com/repos/xbmc/xbmc/issues/comments/308523842","html_url":"https://github.com/xbmc/xbmc/pull/12300#issuecomment-308523842","issue_url":"https://api.github.com/repos/xbmc/xbmc/issues/12300","id":308523842,"user":{"login":"fritsch","id":1089309,"avatar_url":"https://avatars3.githubusercontent.com/u/1089309?v=3","gravatar_id":"","url":"https://api.github.com/users/fritsch","html_url":"https://github.com/fritsch","followers_url":"https://api.github.com/users/fritsch/followers","following_url":"https://api.github.com/users/fritsch/following{/other_user}","gists_url":"https://api.github.com/users/fritsch/gists{/gist_id}","starred_url":"https://api.github.com/users/fritsch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fritsch/subscriptions","organizations_url":"https://api.github.com/users/fritsch/orgs","repos_url":"https://api.github.com/users/fritsch/repos","events_url":"https://api.github.com/users/fritsch/events{/privacy}","received_events_url":"https://api.github.com/users/fritsch/received_events","type":"User","site_admin":false},"created_at":"2017-06-14T18:47:49Z","updated_at":"2017-06-14T18:47:49Z","body":"https://www.youtube.com/watch?v=B0CmilESWms vidibee vidiboo vidiboho"}}
{"actor":{"display_login":"fritsch"}}