{"action":"created","issue":{"url":"https://api.github.com/repos/mamedev/mame/issues/1768","repository_url":"https://api.github.com/repos/mamedev/mame","labels_url":"https://api.github.com/repos/mamedev/mame/issues/1768/labels{/name}","comments_url":"https://api.github.com/repos/mamedev/mame/issues/1768/comments","events_url":"https://api.github.com/repos/mamedev/mame/issues/1768/events","html_url":"https://github.com/mamedev/mame/pull/1768","id":191363147,"number":1768,"title":"frontend: removed redundant 'else' that cause only a code indentation. (nw)","user":{"login":"dankan1890","id":7107228,"avatar_url":"https://avatars.githubusercontent.com/u/7107228?v=3","gravatar_id":"","url":"https://api.github.com/users/dankan1890","html_url":"https://github.com/dankan1890","followers_url":"https://api.github.com/users/dankan1890/followers","following_url":"https://api.github.com/users/dankan1890/following{/other_user}","gists_url":"https://api.github.com/users/dankan1890/gists{/gist_id}","starred_url":"https://api.github.com/users/dankan1890/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dankan1890/subscriptions","organizations_url":"https://api.github.com/users/dankan1890/orgs","repos_url":"https://api.github.com/users/dankan1890/repos","events_url":"https://api.github.com/users/dankan1890/events{/privacy}","received_events_url":"https://api.github.com/users/dankan1890/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-11-23T19:38:35Z","updated_at":"2016-11-23T20:11:52Z","closed_at":"2016-11-23T20:11:52Z","pull_request":{"url":"https://api.github.com/repos/mamedev/mame/pulls/1768","html_url":"https://github.com/mamedev/mame/pull/1768","diff_url":"https://github.com/mamedev/mame/pull/1768.diff","patch_url":"https://github.com/mamedev/mame/pull/1768.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/mamedev/mame/issues/comments/262619225","html_url":"https://github.com/mamedev/mame/pull/1768#issuecomment-262619225","issue_url":"https://api.github.com/repos/mamedev/mame/issues/1768","id":262619225,"user":{"login":"cuavas","id":9678431,"avatar_url":"https://avatars.githubusercontent.com/u/9678431?v=3","gravatar_id":"","url":"https://api.github.com/users/cuavas","html_url":"https://github.com/cuavas","followers_url":"https://api.github.com/users/cuavas/followers","following_url":"https://api.github.com/users/cuavas/following{/other_user}","gists_url":"https://api.github.com/users/cuavas/gists{/gist_id}","starred_url":"https://api.github.com/users/cuavas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cuavas/subscriptions","organizations_url":"https://api.github.com/users/cuavas/orgs","repos_url":"https://api.github.com/users/cuavas/repos","events_url":"https://api.github.com/users/cuavas/events{/privacy}","received_events_url":"https://api.github.com/users/cuavas/received_events","type":"User","site_admin":false},"created_at":"2016-11-23T20:11:52Z","updated_at":"2016-11-23T20:11:52Z","body":"The `else` statements still serve a purpose in guiding the developer showing the conceptual flow of the code.  Using the cheat.cpp change as an example, the code was structured as:\r\n```\r\n    if (is software list)\r\n    {\r\n        software list action\r\n    }\r\n    else\r\n    {\r\n        non software list action\r\n    }\r\n````\r\n\r\nEven if the first block contains an early exit, the code is still arguably easier to follow this way."}}
{"actor":{"display_login":"cuavas"}}