went back and forth on built in classes

decided the way we had it is right because otherwise we end up having to
plan for null parents in the other visitors that have nothing to check
in the built in classes anyways like the Annotating visitor