Clean up QPointer check in comment widget

The QPointer should never become null during exec, as it does not destroy
itself, and does not set WA_DeleteOnClose, so the check is just a
safeguard.

Verify the state just once (and crash in debug builds on failure), and
use QPointer::isNull() to avoid confusing code checkers.