fixed reverse() so it runs in-place instead of as new-returning

also added a reverse() unit test, and updated strings.pde unit test too since it used reverse() wrong