{"action":"created","issue":{"url":"https://api.github.com/repos/insieme/insieme/issues/101","labels_url":"https://api.github.com/repos/insieme/insieme/issues/101/labels{/name}","comments_url":"https://api.github.com/repos/insieme/insieme/issues/101/comments","events_url":"https://api.github.com/repos/insieme/insieme/issues/101/events","html_url":"https://github.com/insieme/insieme/issues/101","id":94306361,"number":101,"title":"Cleanup of code generated from switch cases - remove dead code","user":{"login":"peterz-dps","id":11718093,"avatar_url":"https://avatars.githubusercontent.com/u/11718093?v=3","gravatar_id":"","url":"https://api.github.com/users/peterz-dps","html_url":"https://github.com/peterz-dps","followers_url":"https://api.github.com/users/peterz-dps/followers","following_url":"https://api.github.com/users/peterz-dps/following{/other_user}","gists_url":"https://api.github.com/users/peterz-dps/gists{/gist_id}","starred_url":"https://api.github.com/users/peterz-dps/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peterz-dps/subscriptions","organizations_url":"https://api.github.com/users/peterz-dps/orgs","repos_url":"https://api.github.com/users/peterz-dps/repos","events_url":"https://api.github.com/users/peterz-dps/events{/privacy}","received_events_url":"https://api.github.com/users/peterz-dps/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/insieme/insieme/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/insieme/insieme/labels/frontend","name":"frontend","color":"bfdadc"},{"url":"https://api.github.com/repos/insieme/insieme/labels/minor","name":"minor","color":"bfe5bf"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-07-10T13:22:19Z","updated_at":"2015-12-01T17:25:09Z","closed_at":"2015-12-01T17:25:09Z","body":"The translation of a switch statement often causes the code of multiple cases to end up within each single case - separated by break statements. The code behind each break actually is dead code and could be removed.\r\n\r\nThis should be easily solvable with a pattern matcher - however we must not forget to also look for labels after the break statement - in this case the code might not be dead after all.\r\n\r\nAlso note that the same arguments also holds for continue, return and throw. All of these are normally followed only by dead code."},"comment":{"url":"https://api.github.com/repos/insieme/insieme/issues/comments/161039621","html_url":"https://github.com/insieme/insieme/issues/101#issuecomment-161039621","issue_url":"https://api.github.com/repos/insieme/insieme/issues/101","id":161039621,"user":{"login":"PeterTh","id":1033475,"avatar_url":"https://avatars.githubusercontent.com/u/1033475?v=3","gravatar_id":"","url":"https://api.github.com/users/PeterTh","html_url":"https://github.com/PeterTh","followers_url":"https://api.github.com/users/PeterTh/followers","following_url":"https://api.github.com/users/PeterTh/following{/other_user}","gists_url":"https://api.github.com/users/PeterTh/gists{/gist_id}","starred_url":"https://api.github.com/users/PeterTh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PeterTh/subscriptions","organizations_url":"https://api.github.com/users/PeterTh/orgs","repos_url":"https://api.github.com/users/PeterTh/repos","events_url":"https://api.github.com/users/PeterTh/events{/privacy}","received_events_url":"https://api.github.com/users/PeterTh/received_events","type":"User","site_admin":false},"created_at":"2015-12-01T17:25:09Z","updated_at":"2015-12-01T17:25:09Z","body":"I did this for all common cases, very simply, and with basically 0 performance overhead, directly in the new frontend switch/case handling."}}