I was wondering what should be the focus of review. I thought the focus would have been more on:
- correctness (no syntax or logic error)
- readability (clarity, respect of coding standard ...)
- robustness (ability to handle edge cases)
I've noticed, instead, that the discussion sometimes shifts on the fact that there might be other solutions that might be "better" without pointing to specific flaws of the code under review.
What do you think?