isn't the LJ::load_userpics(\%pic, [ $u, $picid ]); call unneccesary? The %pic is never used in this context - presumably that's only for cases where you need some of the info about teh pic, like the keywords or something similar, and not this case...
Am I confused?