factor out the comparisons from VectorVisitor_Impl so that we don't need special handling of STRSXP
no need for make_visitor
moving VectorVisitorImpl template to its own file