code review modify parameters