passing parameters properly