{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/5605","repository_url":"https://api.github.com/repos/RIOT-OS/RIOT","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/5605/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/5605/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/5605/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/5605","id":163849193,"number":5605,"title":"drivers/dht: fix fake parsing error with DHT22","user":{"login":"MichelRottleuthner","id":20105554,"avatar_url":"https://avatars.githubusercontent.com/u/20105554?v=3","gravatar_id":"","url":"https://api.github.com/users/MichelRottleuthner","html_url":"https://github.com/MichelRottleuthner","followers_url":"https://api.github.com/users/MichelRottleuthner/followers","following_url":"https://api.github.com/users/MichelRottleuthner/following{/other_user}","gists_url":"https://api.github.com/users/MichelRottleuthner/gists{/gist_id}","starred_url":"https://api.github.com/users/MichelRottleuthner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MichelRottleuthner/subscriptions","organizations_url":"https://api.github.com/users/MichelRottleuthner/orgs","repos_url":"https://api.github.com/users/MichelRottleuthner/repos","events_url":"https://api.github.com/users/MichelRottleuthner/events{/privacy}","received_events_url":"https://api.github.com/users/MichelRottleuthner/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Device%20driver","name":"Device driver","color":"aaffff"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Ready%20for%20CI%20build","name":"Ready for CI build","color":"bfdadc"}],"state":"open","locked":false,"assignee":{"login":"PeterKietzmann","id":7765855,"avatar_url":"https://avatars.githubusercontent.com/u/7765855?v=3","gravatar_id":"","url":"https://api.github.com/users/PeterKietzmann","html_url":"https://github.com/PeterKietzmann","followers_url":"https://api.github.com/users/PeterKietzmann/followers","following_url":"https://api.github.com/users/PeterKietzmann/following{/other_user}","gists_url":"https://api.github.com/users/PeterKietzmann/gists{/gist_id}","starred_url":"https://api.github.com/users/PeterKietzmann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PeterKietzmann/subscriptions","organizations_url":"https://api.github.com/users/PeterKietzmann/orgs","repos_url":"https://api.github.com/users/PeterKietzmann/repos","events_url":"https://api.github.com/users/PeterKietzmann/events{/privacy}","received_events_url":"https://api.github.com/users/PeterKietzmann/received_events","type":"User","site_admin":false},"assignees":[{"login":"PeterKietzmann","id":7765855,"avatar_url":"https://avatars.githubusercontent.com/u/7765855?v=3","gravatar_id":"","url":"https://api.github.com/users/PeterKietzmann","html_url":"https://github.com/PeterKietzmann","followers_url":"https://api.github.com/users/PeterKietzmann/followers","following_url":"https://api.github.com/users/PeterKietzmann/following{/other_user}","gists_url":"https://api.github.com/users/PeterKietzmann/gists{/gist_id}","starred_url":"https://api.github.com/users/PeterKietzmann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PeterKietzmann/subscriptions","organizations_url":"https://api.github.com/users/PeterKietzmann/orgs","repos_url":"https://api.github.com/users/PeterKietzmann/repos","events_url":"https://api.github.com/users/PeterKietzmann/events{/privacy}","received_events_url":"https://api.github.com/users/PeterKietzmann/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2016-07-05T13:21:48Z","updated_at":"2016-07-05T15:51:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/5605","html_url":"https://github.com/RIOT-OS/RIOT/pull/5605","diff_url":"https://github.com/RIOT-OS/RIOT/pull/5605.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/5605.patch"},"body":"The switch-case in [dht.c](https://github.com/RIOT-OS/RIOT/blob/master/drivers/dht/dht.c#L146) that interprets which sensor type is in use lacks a **break;** statement at the end of the DHT22 case. This leads to a fall through to the default case which then always returns -2 (parsing error) to the caller of the function. \r\n@PeterKietzmann we talked about this bug some time ago. Since it's obvious what needs to be fixed and it's only one line I created a PR right away instead of raising an issue to keep it simple."},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/230519512","html_url":"https://github.com/RIOT-OS/RIOT/pull/5605#issuecomment-230519512","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/5605","id":230519512,"user":{"login":"PeterKietzmann","id":7765855,"avatar_url":"https://avatars.githubusercontent.com/u/7765855?v=3","gravatar_id":"","url":"https://api.github.com/users/PeterKietzmann","html_url":"https://github.com/PeterKietzmann","followers_url":"https://api.github.com/users/PeterKietzmann/followers","following_url":"https://api.github.com/users/PeterKietzmann/following{/other_user}","gists_url":"https://api.github.com/users/PeterKietzmann/gists{/gist_id}","starred_url":"https://api.github.com/users/PeterKietzmann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PeterKietzmann/subscriptions","organizations_url":"https://api.github.com/users/PeterKietzmann/orgs","repos_url":"https://api.github.com/users/PeterKietzmann/repos","events_url":"https://api.github.com/users/PeterKietzmann/events{/privacy}","received_events_url":"https://api.github.com/users/PeterKietzmann/received_events","type":"User","site_admin":false},"created_at":"2016-07-05T15:51:39Z","updated_at":"2016-07-05T15:51:39Z","body":"ACK and go"}}
{"actor":{"display_login":"PeterKietzmann"}}