The problem with doing it this way is that you're going to be running "my_sort" many many times, introducing a lot of overhead.

That's what the Schwartzian Transform, GRT, or Orcish Maneuvre save you from; they only compute each key once.

George_Sherston's original solution also avoided recomputing keys unnecessarily, by storing them in ->{Sort} beforehand.
--
Mike