In our context, ``$oldname`` is the new cache file while ``$newname`` is the old cache file. This was a bit confusing when I read the method this morning.

How about using ``$source`` and ``$target`` instead?