Checking for null would indeed be "safer", however the code that calls this is really "infrastructure" code and will always pass in a valid value for `pResult`, which is why I didn't really bother. It is one of those minor code size optimisations. If you're more comfortable with the NULL check, I'm happy to add it back in for the sake of a couple of extra bytes.