change unused return values to void