{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/5929","repository_url":"https://api.github.com/repos/RIOT-OS/RIOT","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/5929/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/5929/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/5929/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/5929","id":182378566,"number":5929,"title":"sock: change \"no timeout\" value from 0 to UINT32_MAX","user":{"login":"kaspar030","id":4679640,"avatar_url":"https://avatars.githubusercontent.com/u/4679640?v=3","gravatar_id":"","url":"https://api.github.com/users/kaspar030","html_url":"https://github.com/kaspar030","followers_url":"https://api.github.com/users/kaspar030/followers","following_url":"https://api.github.com/users/kaspar030/following{/other_user}","gists_url":"https://api.github.com/users/kaspar030/gists{/gist_id}","starred_url":"https://api.github.com/users/kaspar030/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kaspar030/subscriptions","organizations_url":"https://api.github.com/users/kaspar030/orgs","repos_url":"https://api.github.com/users/kaspar030/repos","events_url":"https://api.github.com/users/kaspar030/events{/privacy}","received_events_url":"https://api.github.com/users/kaspar030/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/API%20CHANGE","name":"API CHANGE","color":"e11d21"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/minor","name":"minor","color":"f7c6c7"}],"state":"open","locked":false,"assignee":{"login":"miri64","id":675644,"avatar_url":"https://avatars.githubusercontent.com/u/675644?v=3","gravatar_id":"","url":"https://api.github.com/users/miri64","html_url":"https://github.com/miri64","followers_url":"https://api.github.com/users/miri64/followers","following_url":"https://api.github.com/users/miri64/following{/other_user}","gists_url":"https://api.github.com/users/miri64/gists{/gist_id}","starred_url":"https://api.github.com/users/miri64/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miri64/subscriptions","organizations_url":"https://api.github.com/users/miri64/orgs","repos_url":"https://api.github.com/users/miri64/repos","events_url":"https://api.github.com/users/miri64/events{/privacy}","received_events_url":"https://api.github.com/users/miri64/received_events","type":"User","site_admin":false},"assignees":[{"login":"miri64","id":675644,"avatar_url":"https://avatars.githubusercontent.com/u/675644?v=3","gravatar_id":"","url":"https://api.github.com/users/miri64","html_url":"https://github.com/miri64","followers_url":"https://api.github.com/users/miri64/followers","following_url":"https://api.github.com/users/miri64/following{/other_user}","gists_url":"https://api.github.com/users/miri64/gists{/gist_id}","starred_url":"https://api.github.com/users/miri64/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miri64/subscriptions","organizations_url":"https://api.github.com/users/miri64/orgs","repos_url":"https://api.github.com/users/miri64/repos","events_url":"https://api.github.com/users/miri64/events{/privacy}","received_events_url":"https://api.github.com/users/miri64/received_events","type":"User","site_admin":false}],"milestone":null,"comments":5,"created_at":"2016-10-11T21:25:33Z","updated_at":"2016-10-12T07:44:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/5929","html_url":"https://github.com/RIOT-OS/RIOT/pull/5929","diff_url":"https://github.com/RIOT-OS/RIOT/pull/5929.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/5929.patch"},"body":"IMHO specifying a timeout of \"0\" for \"don't wait at all\" makes much more sense."},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/253142340","html_url":"https://github.com/RIOT-OS/RIOT/pull/5929#issuecomment-253142340","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/5929","id":253142340,"user":{"login":"miri64","id":675644,"avatar_url":"https://avatars.githubusercontent.com/u/675644?v=3","gravatar_id":"","url":"https://api.github.com/users/miri64","html_url":"https://github.com/miri64","followers_url":"https://api.github.com/users/miri64/followers","following_url":"https://api.github.com/users/miri64/following{/other_user}","gists_url":"https://api.github.com/users/miri64/gists{/gist_id}","starred_url":"https://api.github.com/users/miri64/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miri64/subscriptions","organizations_url":"https://api.github.com/users/miri64/orgs","repos_url":"https://api.github.com/users/miri64/repos","events_url":"https://api.github.com/users/miri64/events{/privacy}","received_events_url":"https://api.github.com/users/miri64/received_events","type":"User","site_admin":false},"created_at":"2016-10-12T07:44:55Z","updated_at":"2016-10-12T07:44:55Z","body":">> IMHO specifying a timeout of \"0\" for \"don't wait at all\" makes much more sense.\r\n\r\nI should read more...\r\n\r\nNevertheless, I still don't think it's a good idea. `0` means to me no timeout at all, not 0 sec timeout. Also, though my assembly is quite rusty, I think `if (t == 0)` is translated into fewer instructions than `if (t < UINT_MAX32)`."}}
{"actor":{"display_login":"miri64"}}