IMO we shouldn't return - if the method is not intended to return anything.  (prefer the previous `if ` structure)