I understand your intention, but there's no value in returning a value here as nothing checks the return value of do_history