do we need to also extend the other ExpressionTo*Visitor to handle NullLiterals?