The only minor suggestion I have is to have a local variable to replace the two places we do the above mod.