prefer a return value over void