why do code reviews always devolve into bike shedding over variable names when there are actual logic errors to discuss