{"comment":{"url":"https://api.github.com/repos/Glenn-1990/xbmc/comments/10450916","html_url":"https://github.com/Glenn-1990/xbmc/commit/66fc8eda3eee815df8f8d2d1b919bbfe4a65163c#commitcomment-10450916","id":10450916,"user":{"login":"ksooo","id":3226626,"avatar_url":"https://avatars.githubusercontent.com/u/3226626?v=3","gravatar_id":"","url":"https://api.github.com/users/ksooo","html_url":"https://github.com/ksooo","followers_url":"https://api.github.com/users/ksooo/followers","following_url":"https://api.github.com/users/ksooo/following{/other_user}","gists_url":"https://api.github.com/users/ksooo/gists{/gist_id}","starred_url":"https://api.github.com/users/ksooo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ksooo/subscriptions","organizations_url":"https://api.github.com/users/ksooo/orgs","repos_url":"https://api.github.com/users/ksooo/repos","events_url":"https://api.github.com/users/ksooo/events{/privacy}","received_events_url":"https://api.github.com/users/ksooo/received_events","type":"User","site_admin":false},"position":438,"line":391,"path":"xbmc/pvr/dialogs/GUIDialogPVRTimerSettings.cpp","commit_id":"66fc8eda3eee815df8f8d2d1b919bbfe4a65163c","created_at":"2015-03-28T17:41:00Z","updated_at":"2015-03-28T17:41:00Z","body":"Shouldn't different members be used for the type and the weekdays, to make clear what actually is contained? Sure, both are ints, but semantically the type is completely different from scheduled weekdays."}}