{"action":"created","issue":{"url":"https://api.github.com/repos/krs-world/bridges-cxx/issues/27","labels_url":"https://api.github.com/repos/krs-world/bridges-cxx/issues/27/labels{/name}","comments_url":"https://api.github.com/repos/krs-world/bridges-cxx/issues/27/comments","events_url":"https://api.github.com/repos/krs-world/bridges-cxx/issues/27/events","html_url":"https://github.com/krs-world/bridges-cxx/issues/27","id":115783675,"number":27,"title":"Unneeded try/catches in Bridges.h","user":{"login":"dcarmer","id":8569832,"avatar_url":"https://avatars.githubusercontent.com/u/8569832?v=3","gravatar_id":"","url":"https://api.github.com/users/dcarmer","html_url":"https://github.com/dcarmer","followers_url":"https://api.github.com/users/dcarmer/followers","following_url":"https://api.github.com/users/dcarmer/following{/other_user}","gists_url":"https://api.github.com/users/dcarmer/gists{/gist_id}","starred_url":"https://api.github.com/users/dcarmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dcarmer/subscriptions","organizations_url":"https://api.github.com/users/dcarmer/orgs","repos_url":"https://api.github.com/users/dcarmer/repos","events_url":"https://api.github.com/users/dcarmer/events{/privacy}","received_events_url":"https://api.github.com/users/dcarmer/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/krs-world/bridges-cxx/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/krs-world/bridges-cxx/labels/pedantic","name":"pedantic","color":"bfe5bf"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-11-09T03:15:43Z","updated_at":"2015-11-17T01:05:07Z","closed_at":"2015-11-17T01:05:07Z","body":"In Bridges.h several functions contain try/catch blocks for a manual throw. With the if block already determining if the error is present, it seems redundant to place it all in an additional try/catch block. Anything that you have after the throw could just be put in an else block instead.  \r\n\r\nThe try/catch blocks should be removed, with the catch statement(s) put in the __if__ block and with the remainder of the try statement(s) put in the __else__ block."},"comment":{"url":"https://api.github.com/repos/krs-world/bridges-cxx/issues/comments/157226246","html_url":"https://github.com/krs-world/bridges-cxx/issues/27#issuecomment-157226246","issue_url":"https://api.github.com/repos/krs-world/bridges-cxx/issues/27","id":157226246,"user":{"login":"dcarmer","id":8569832,"avatar_url":"https://avatars.githubusercontent.com/u/8569832?v=3","gravatar_id":"","url":"https://api.github.com/users/dcarmer","html_url":"https://github.com/dcarmer","followers_url":"https://api.github.com/users/dcarmer/followers","following_url":"https://api.github.com/users/dcarmer/following{/other_user}","gists_url":"https://api.github.com/users/dcarmer/gists{/gist_id}","starred_url":"https://api.github.com/users/dcarmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dcarmer/subscriptions","organizations_url":"https://api.github.com/users/dcarmer/orgs","repos_url":"https://api.github.com/users/dcarmer/repos","events_url":"https://api.github.com/users/dcarmer/events{/privacy}","received_events_url":"https://api.github.com/users/dcarmer/received_events","type":"User","site_admin":false},"created_at":"2015-11-17T01:05:07Z","updated_at":"2015-11-17T01:05:07Z","body":"Looks good"}}