use parameter not constant