legacy vector min max should take const args

The free min and max function for the legacy implementation of VectorT (VectorT.hh) only accept mutable (non-const) arguments. When you call these functions with const arguments, the std::min / std::max implementation will match and return the lexicographically smaller argument.