Не, это развод токсичной культуры. Неконструктив в ревью - признак профессиональной незрелости
Коллеги, в какой момент мы стали такими нежными? Ну, я понимаю, буллинг - это оскорбления в адрес человека, и никогда, ни в какой ситуации не перехожу на личности... Но... В какой момент стало нельзя называть откровенно плохой код плохим? Заметьте, не человека, который его написал, а сам код?
Конкретно в этой ситуации - передать в вызов метода пустую структуру, чтобы потом внутри метода проверить, что она пустая и вернуть ошибку "она пустая" до записи в базу. Ну, собственно, нас спросили, почему оно не работает - я сказал "косяк здесь".
Ну, я, конечно, понимаю, что надо всенепременно за что-то похвалить человека, чтобы он не расстраивался... Но я не нашел, за что. Там var err error в глобальном скоупе, т.е. на все хендлеры одна переменная, хранящая ошибку... Ну, т.е., исправление в указанном мной месте глобально проблему не решит, там нужно прямо все переписывать, о чем я и говорю "весь код косяк".
Если человек воспринимает это как личное оскорбление, непереносимую обиду и повод все бросить... Ну, не знаю, а работать потом как?