change return type to void as its not using int