{"comment":{"url":"https://api.github.com/repos/pguyot/Einstein/comments/11502624","html_url":"https://github.com/pguyot/Einstein/commit/e1b82ff271581967efb997383de2bb0b0a5b6f22#commitcomment-11502624","id":11502624,"user":{"login":"pguyot","id":168407,"avatar_url":"https://avatars.githubusercontent.com/u/168407?v=3","gravatar_id":"","url":"https://api.github.com/users/pguyot","html_url":"https://github.com/pguyot","followers_url":"https://api.github.com/users/pguyot/followers","following_url":"https://api.github.com/users/pguyot/following{/other_user}","gists_url":"https://api.github.com/users/pguyot/gists{/gist_id}","starred_url":"https://api.github.com/users/pguyot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pguyot/subscriptions","organizations_url":"https://api.github.com/users/pguyot/orgs","repos_url":"https://api.github.com/users/pguyot/repos","events_url":"https://api.github.com/users/pguyot/events{/privacy}","received_events_url":"https://api.github.com/users/pguyot/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"e1b82ff271581967efb997383de2bb0b0a5b6f22","created_at":"2015-06-03T16:05:44Z","updated_at":"2015-06-03T16:05:44Z","body":"Actually, I believe *this* can definitely be null in C++ and that calling a method on a null pointer can even be valid. See : http://www.open-std.org/jtc1/sc22/wg21/docs/cwg_closed.html#315 and http://stackoverflow.com/questions/28482809/c-access-static-members-using-null-pointer for further discussion.\r\n\r\nYet, I agree that the code was inconsistent, sometimes checking if mLog is null at the caller level."}}