add test for side-effect (param modified by reference)
minor: todo comment, whsp