Hacker News: silva97

New comment by silva97 in "The primary purpose of code review is to find code that will be hard to maintain"

silva97 — Thu, 02 Jul 2026 19:24:57 +0000

To criticize is, basically, to say what should have been different. It is in the past, yet it is still about what should be changed.

> making code more readable is a "change." I don't know what you're trying to say here.

Yes and this is exactly what I am trying to say. The author limited what is the purpose of a code review like it's has really this limitation but, in practice, the code review is widely about what you think it should changed on the code by multiple reasons: readability, manutenibility, performance, security, bugs, use cases, test coverage etc.

Making code more readable is just one of the reasons why you want the code to be changed on a pull request, it's not the primary purpose.

New comment by silva97 in "The primary purpose of code review is to find code that will be hard to maintain"

silva97 — Thu, 02 Jul 2026 14:35:48 +0000

The author seens to misunderstand the purpose of code review. The purpose is, literally, review the code. Review means basically to think/talk about something again in order to make or not changes on it. When you review something (including code), you are basically asking for yourself: "Should it be changed or it's okay to stay like this?"

In order words, the purpose of code review is to or not ask for changes on the code.

Linux Kernel vulnerability research toolkit

silva97 — Mon, 15 Jun 2026 18:55:38 +0000

Article URL: https://github.com/freedev-org/kernelvr

Comments URL: https://news.ycombinator.com/item?id=48545506

Points: 3

# Comments: 0