{"comment":{"url":"https://api.github.com/repos/kindlycat/awesome/comments/14500384","html_url":"https://github.com/kindlycat/awesome/commit/043c60f389fd88b4fcd5b9036d10817844fb4669#commitcomment-14500384","id":14500384,"user":{"login":"psychon","id":89482,"avatar_url":"https://avatars.githubusercontent.com/u/89482?v=3","gravatar_id":"","url":"https://api.github.com/users/psychon","html_url":"https://github.com/psychon","followers_url":"https://api.github.com/users/psychon/followers","following_url":"https://api.github.com/users/psychon/following{/other_user}","gists_url":"https://api.github.com/users/psychon/gists{/gist_id}","starred_url":"https://api.github.com/users/psychon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/psychon/subscriptions","organizations_url":"https://api.github.com/users/psychon/orgs","repos_url":"https://api.github.com/users/psychon/repos","events_url":"https://api.github.com/users/psychon/events{/privacy}","received_events_url":"https://api.github.com/users/psychon/received_events","type":"User","site_admin":false},"position":102,"line":102,"path":"lib/awful/widget/slider.lua","commit_id":"043c60f389fd88b4fcd5b9036d10817844fb4669","created_at":"2015-11-19T14:07:13Z","updated_at":"2015-11-19T14:07:13Z","body":"What's the point of this cache? It doesn't seem to cache anything that is hard to compute, so this shouldn't be for speed reasons. Also, the cache is wrong, because if the width/height is changed, the cache would need to be invalidated (imaging the widget being visible at two different places in two different sizes)."}}