do we need a tmp var here, or could we move this inline to the method call?