{"action":"created","issue":{"url":"https://api.github.com/repos/dlang/phobos/issues/4827","repository_url":"https://api.github.com/repos/dlang/phobos","labels_url":"https://api.github.com/repos/dlang/phobos/issues/4827/labels{/name}","comments_url":"https://api.github.com/repos/dlang/phobos/issues/4827/comments","events_url":"https://api.github.com/repos/dlang/phobos/issues/4827/events","html_url":"https://github.com/dlang/phobos/pull/4827","id":180433195,"number":4827,"title":"Restrict length","user":{"login":"andralex","id":566679,"avatar_url":"https://avatars.githubusercontent.com/u/566679?v=3","gravatar_id":"","url":"https://api.github.com/users/andralex","html_url":"https://github.com/andralex","followers_url":"https://api.github.com/users/andralex/followers","following_url":"https://api.github.com/users/andralex/following{/other_user}","gists_url":"https://api.github.com/users/andralex/gists{/gist_id}","starred_url":"https://api.github.com/users/andralex/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andralex/subscriptions","organizations_url":"https://api.github.com/users/andralex/orgs","repos_url":"https://api.github.com/users/andralex/repos","events_url":"https://api.github.com/users/andralex/events{/privacy}","received_events_url":"https://api.github.com/users/andralex/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-10-01T02:30:14Z","updated_at":"2016-11-12T17:55:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dlang/phobos/pulls/4827","html_url":"https://github.com/dlang/phobos/pull/4827","diff_url":"https://github.com/dlang/phobos/pull/4827.diff","patch_url":"https://github.com/dlang/phobos/pull/4827.patch"},"body":"Fixes https://issues.dlang.org/show_bug.cgi?id=16566. `hasLength` was in fact intended to allow only integrals, but its code predates `alias this` and we didn't update it.\n\nThroughout Phobos we use interchangeably `auto s = r.length;`, `size_t s = r.length;`, cache the length of a range in a variable, or just use it as a `size_t`. Supporting other types than `size_t` meaningfully would put too much burden on the definition and the implementation.\n"},"comment":{"url":"https://api.github.com/repos/dlang/phobos/issues/comments/260137611","html_url":"https://github.com/dlang/phobos/pull/4827#issuecomment-260137611","issue_url":"https://api.github.com/repos/dlang/phobos/issues/4827","id":260137611,"user":{"login":"andralex","id":566679,"avatar_url":"https://avatars.githubusercontent.com/u/566679?v=3","gravatar_id":"","url":"https://api.github.com/users/andralex","html_url":"https://github.com/andralex","followers_url":"https://api.github.com/users/andralex/followers","following_url":"https://api.github.com/users/andralex/following{/other_user}","gists_url":"https://api.github.com/users/andralex/gists{/gist_id}","starred_url":"https://api.github.com/users/andralex/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andralex/subscriptions","organizations_url":"https://api.github.com/users/andralex/orgs","repos_url":"https://api.github.com/users/andralex/repos","events_url":"https://api.github.com/users/andralex/events{/privacy}","received_events_url":"https://api.github.com/users/andralex/received_events","type":"User","site_admin":false},"created_at":"2016-11-12T17:55:17Z","updated_at":"2016-11-12T17:55:17Z","body":"I plan to be more aggressive about this: if an artifact defines `length` to have type `ulong` on a 32-bit system, I'll yield a deprecation message. @MartinNowak does this work for you?"}}
{"actor":{"display_login":"andralex"}}