Cautiously avoiding the scoping issue that was discussed on the list, I thought these other changes needed to go in; Namely that we don't need to check if the brigade is empty twice in the loop, just once. Added a comment for good measure.