I will take a wild blow in the dark and say that people fall into two camps:
those who delete the line consider it redundant, and all such code should be removed for brevity
those who add the string think that it makes the return value understandable and unambiguous for smaller coders.
Personally, I would always like to write a meaningful return in main in my production code (if only because my production main tends to also contain code codes that ultimately return something other than 0 , usually in exception handlers) , although I would not bother with the trivial main , which never returns anything; for example, I donβt think I have ever done this, for example, in the Coliru column for a demonstration.
Some will say that it is absurd to change the code base to switch between these two states, that the arguments are very weak in the great scheme of things, and that such a personal choice is not worth the risk of introducing errors.
But I would say that it almost entirely depends on your environment. If you are halfway to the release cycle, of course, you are going to improve the improvements in code content and style settings: this is the best time to avoid the accumulation of technical debt, and you absolutely want to do it. But , if you plan to make this change directly on the production server or in version control one week before this big release, you are crazy.
(I hope your politicians prevent this kind of madness anyway. The code freezes, right? No production changes, right?)
So, although it goes without saying that the main choice is very subjective , we can quantify the risk / benefit to ensure such a choice after the fact.
But don't think about real life, but what about a code review? Well, I have no idea; you should ask them. Personally, with these specific examples, I probably would have deleted it too, if only with a written warning that it was a pure choice of style. Whether pure style changes are appropriate for Code Review is the question for Meta Code Review.