{"action":"created","issue":{"url":"https://api.github.com/repos/MvvmCross/MvvmCross/issues/1004","labels_url":"https://api.github.com/repos/MvvmCross/MvvmCross/issues/1004/labels{/name}","comments_url":"https://api.github.com/repos/MvvmCross/MvvmCross/issues/1004/comments","events_url":"https://api.github.com/repos/MvvmCross/MvvmCross/issues/1004/events","html_url":"https://github.com/MvvmCross/MvvmCross/issues/1004","id":75573687,"number":1004,"title":"Warning issue around MvxAdapter code","user":{"login":"monesprit54","id":1914264,"avatar_url":"https://avatars.githubusercontent.com/u/1914264?v=3","gravatar_id":"","url":"https://api.github.com/users/monesprit54","html_url":"https://github.com/monesprit54","followers_url":"https://api.github.com/users/monesprit54/followers","following_url":"https://api.github.com/users/monesprit54/following{/other_user}","gists_url":"https://api.github.com/users/monesprit54/gists{/gist_id}","starred_url":"https://api.github.com/users/monesprit54/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/monesprit54/subscriptions","organizations_url":"https://api.github.com/users/monesprit54/orgs","repos_url":"https://api.github.com/users/monesprit54/repos","events_url":"https://api.github.com/users/monesprit54/events{/privacy}","received_events_url":"https://api.github.com/users/monesprit54/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-05-12T11:43:35Z","updated_at":"2015-05-12T16:51:29Z","closed_at":null,"body":"In MvxAdapter, MvvmCross checks whether an ItemSource is IList or not, and says\r\n\"Binding to IEnumerable rather than IList - this can be inefficient, especially for large lists\"\r\n\r\nBut I think that it should be vice versa like..\r\n\"Binding to IList rather than IEnumerable - this can be inefficient, especially for large lists\"\r\nsince IList is preferred to IEnumerable as I understand? \r\n\r\nhttps://github.com/MvvmCross/MvvmCross/blob/3.2/Cirrious/Cirrious.MvvmCross.Binding.Droid/Views/MvxAdapter.cs#L140\r\n\r\nPlease check this."},"comment":{"url":"https://api.github.com/repos/MvvmCross/MvvmCross/issues/comments/101346085","html_url":"https://github.com/MvvmCross/MvvmCross/issues/1004#issuecomment-101346085","issue_url":"https://api.github.com/repos/MvvmCross/MvvmCross/issues/1004","id":101346085,"user":{"login":"monesprit54","id":1914264,"avatar_url":"https://avatars.githubusercontent.com/u/1914264?v=3","gravatar_id":"","url":"https://api.github.com/users/monesprit54","html_url":"https://github.com/monesprit54","followers_url":"https://api.github.com/users/monesprit54/followers","following_url":"https://api.github.com/users/monesprit54/following{/other_user}","gists_url":"https://api.github.com/users/monesprit54/gists{/gist_id}","starred_url":"https://api.github.com/users/monesprit54/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/monesprit54/subscriptions","organizations_url":"https://api.github.com/users/monesprit54/orgs","repos_url":"https://api.github.com/users/monesprit54/repos","events_url":"https://api.github.com/users/monesprit54/events{/privacy}","received_events_url":"https://api.github.com/users/monesprit54/received_events","type":"User","site_admin":false},"created_at":"2015-05-12T16:51:29Z","updated_at":"2015-05-12T16:51:29Z","body":"Knew the idea that IList is better than IEnumerable since if you use a class implementing IEnumerable you gotta call \"MoveNext()\" so many times (n times from the head) if you wanna see n-th item. Instead, when you use a descendant of IList as an ItemSource you can just read the value since IList let you access the item randomly.\r\n\r\nThe wording \"Binding to IEnumerable rather than IList\" means you should bind to \"descendants of IEnumerable\" which is the exact opposite of the wording it intended to be, I guess.\r\n\r\nChecks the following link\r\nhttp://en.wiktionary.org/wiki/rather_than"}}