{"action":"created","issue":{"url":"https://api.github.com/repos/godotengine/godot/issues/55","labels_url":"https://api.github.com/repos/godotengine/godot/issues/55/labels{/name}","comments_url":"https://api.github.com/repos/godotengine/godot/issues/55/comments","events_url":"https://api.github.com/repos/godotengine/godot/issues/55/events","html_url":"https://github.com/godotengine/godot/issues/55","id":27519125,"number":55,"title":"Remove unnecessary null pointer checks","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":[{"url":"https://api.github.com/repos/godotengine/godot/labels/enhancement","name":"enhancement","color":"44bbff"},{"url":"https://api.github.com/repos/godotengine/godot/labels/junior%20job","name":"junior job","color":"00eeee"},{"url":"https://api.github.com/repos/godotengine/godot/labels/topic:core","name":"topic:core","color":"ffdd44"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2014-02-13T13:53:16Z","updated_at":"2016-01-13T14:30:56Z","closed_at":null,"body":"[An extra null pointer check is not needed in functions](http://dietmar-kuehl.de/mirror/c++-faq/freestore-mgmt.html#faq-16.8 \"Do I need to check for NULL before delete p?\") like the following.\r\n* [OSNacl::finalize_core](https://github.com/okamstudio/godot/blob/0b806ee0fc9097fa7bda7ac0109191c9c5e0a1ac/platform/nacl/os_nacl.cpp#L143 \"OSNacl::finalize_core function\")\r\n* [OS_Unix::finalize_core](https://github.com/okamstudio/godot/blob/0b806ee0fc9097fa7bda7ac0109191c9c5e0a1ac/drivers/unix/os_unix.cpp#L127 \"OS_Unix::finalize_core function\")\r\n* [OS_Windows::finalize_core](https://github.com/okamstudio/godot/blob/0b806ee0fc9097fa7bda7ac0109191c9c5e0a1ac/platform/windows/os_windows.cpp#L1123 \"OS_Windows::finalize_core function\")"},"comment":{"url":"https://api.github.com/repos/godotengine/godot/issues/comments/171307751","html_url":"https://github.com/godotengine/godot/issues/55#issuecomment-171307751","issue_url":"https://api.github.com/repos/godotengine/godot/issues/55","id":171307751,"user":{"login":"trollworkout","id":6987027,"avatar_url":"https://avatars.githubusercontent.com/u/6987027?v=3","gravatar_id":"","url":"https://api.github.com/users/trollworkout","html_url":"https://github.com/trollworkout","followers_url":"https://api.github.com/users/trollworkout/followers","following_url":"https://api.github.com/users/trollworkout/following{/other_user}","gists_url":"https://api.github.com/users/trollworkout/gists{/gist_id}","starred_url":"https://api.github.com/users/trollworkout/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/trollworkout/subscriptions","organizations_url":"https://api.github.com/users/trollworkout/orgs","repos_url":"https://api.github.com/users/trollworkout/repos","events_url":"https://api.github.com/users/trollworkout/events{/privacy}","received_events_url":"https://api.github.com/users/trollworkout/received_events","type":"User","site_admin":false},"created_at":"2016-01-13T14:30:56Z","updated_at":"2016-01-13T14:30:56Z","body":"@elfring  see here \r\nhttp://www.cplusplus.com/reference/new/operator%20delete/\r\n\r\nreturns nothing\r\n\r\nYou could be sending null pointers to delete and would not even know it.  \r\n\r\nC++ devs say is useful tool in preventing certain bugs from seeping in"}}