Don't return value rather than always returning true.

addInputElement() never fails, and no one checks its return value
except tests. Let's simplify the signature.
Fix leak in ObjCSelfInitChecker, found by LSan.

BugReport doesn't take ownership of the bug type, so let the checker own the
the bug type.  (Requires making the bug type mutable, which is icky, but which
is also what other checkers do.)