change return type to void