params shouldn't have been mutable