{"action":"created","issue":{"url":"https://api.github.com/repos/Croteam-official/Serious-Engine/issues/13","repository_url":"https://api.github.com/repos/Croteam-official/Serious-Engine","labels_url":"https://api.github.com/repos/Croteam-official/Serious-Engine/issues/13/labels{/name}","comments_url":"https://api.github.com/repos/Croteam-official/Serious-Engine/issues/13/comments","events_url":"https://api.github.com/repos/Croteam-official/Serious-Engine/issues/13/events","html_url":"https://github.com/Croteam-official/Serious-Engine/issues/13","id":140415653,"number":13,"title":"Completion of error handling","user":{"login":"elfring","id":660477,"avatar_url":"https://avatars.githubusercontent.com/u/660477?v=3","gravatar_id":"","url":"https://api.github.com/users/elfring","html_url":"https://github.com/elfring","followers_url":"https://api.github.com/users/elfring/followers","following_url":"https://api.github.com/users/elfring/following{/other_user}","gists_url":"https://api.github.com/users/elfring/gists{/gist_id}","starred_url":"https://api.github.com/users/elfring/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/elfring/subscriptions","organizations_url":"https://api.github.com/users/elfring/orgs","repos_url":"https://api.github.com/users/elfring/repos","events_url":"https://api.github.com/users/elfring/events{/privacy}","received_events_url":"https://api.github.com/users/elfring/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-03-12T20:47:11Z","updated_at":"2016-03-12T21:01:11Z","closed_at":null,"body":"Would you like to add more error handling for return values from functions like the following?\r\n- [malloc](http://pubs.opengroup.org/onlinepubs/9699919799/functions/malloc.html \"Memory allocation\") ⇒ [stradd](https://github.com/Croteam-official/Serious-Engine/blob/c9fe415f2cddba5ef01c90e0943189b96b0d19b6/Sources/Ecc/Main.cpp#L41)\r\n- [strdup](http://pubs.opengroup.org/onlinepubs/9699919799/functions/strdup.html \"Duplicate a string.\") ⇒ [ShellPushBuffer](https://github.com/Croteam-official/Serious-Engine/blob/c9fe415f2cddba5ef01c90e0943189b96b0d19b6/Sources/Engine/Base/Shell.cpp#L58)"},"comment":{"url":"https://api.github.com/repos/Croteam-official/Serious-Engine/issues/comments/195808551","html_url":"https://github.com/Croteam-official/Serious-Engine/issues/13#issuecomment-195808551","issue_url":"https://api.github.com/repos/Croteam-official/Serious-Engine/issues/13","id":195808551,"user":{"login":"SLAwww","id":17743522,"avatar_url":"https://avatars.githubusercontent.com/u/17743522?v=3","gravatar_id":"","url":"https://api.github.com/users/SLAwww","html_url":"https://github.com/SLAwww","followers_url":"https://api.github.com/users/SLAwww/followers","following_url":"https://api.github.com/users/SLAwww/following{/other_user}","gists_url":"https://api.github.com/users/SLAwww/gists{/gist_id}","starred_url":"https://api.github.com/users/SLAwww/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SLAwww/subscriptions","organizations_url":"https://api.github.com/users/SLAwww/orgs","repos_url":"https://api.github.com/users/SLAwww/repos","events_url":"https://api.github.com/users/SLAwww/events{/privacy}","received_events_url":"https://api.github.com/users/SLAwww/received_events","type":"User","site_admin":false},"created_at":"2016-03-12T21:01:11Z","updated_at":"2016-03-12T21:01:11Z","body":"This is a good idea, we should have error handling there. The rest of the engine sources use functions from Engine/Base/Memory.h with error handling, but ECC does not link to anything else so it needs separate error handling."}}