I tend to agree with you.
But from earlier code reviews I got the impression that in general one doesn't return "early" at the end.
Same with returning void in earlier PRs.
Any more opinions on that?

It sure is a good thing to have it always return for consistency and avoiding copy-and-paste-errors.