How often is this called? Can it be omitted? It seems like if you really want to take ownership of it then you should call `into_vec`