choose a better name for parameter s