Shouldn't this have a return type?