munt/munt — pull request #19 by lordhoto (2014-07-10 21:06:55+00:00)
Title: Remove incorrect NULL checks.

In well-defined C++ code references can never be bound to dereferenced NULL
pointers. Thus, compilers can assume that the address a reference points at is
always non-NULL. The reason is that if this happens, the behavior is undefined
from now on. Thus, the compiler can do as it pleases. As a result, compilers
might simply remove checks like the ones removed here.

clang's "-Wtautological-undefined-compare" warning option can be used to spot
such issues.

There might be a greater issue in the underlying code design here. Thus, now some supposedly important check might be missing even if the compiler decides not to remove it. So, it's probably best to see whether further changes should be made to adjust for this.