Only a small thing, but I would generally not assign to a function input parameter (should be final in the signature)