nice! how about naming the arguments something a little more descriptive