Block for quick backswitches use user_load in cycle just tot get uid and name.
It's bad practice and have bad perfomance impact.
I propose use single SQL query to fetch this data at once.
While working on another module, I noticed that Masquerade does this a lot. Is there a reason? If not, I'll happily provide a patch to do this module-wide (or as much as possible).
This is a really annoying but required to allow modules a-la realname to work - override a username
Suppose we needs more reviews
Drupal is a registered trademark of Dries Buytaert.