Ignore exceptions is always a bad bad idea, as well as ignoring data returned by functions. I really don't understand people who do that.
I do: manager demands quick fix. This is how this happened. Crash-Fix-NOW!
I only do code reviews nowadays, because of my health (nothing to worry about), but this wasn't an obvious one at first sight.
There are several tens of lines in between and generally well written.
I agree, but pressure can force people into making mistakes.
Every programmer should realise that....
Even if you tell the manager this was a quick fix, you won't get the time to fix it properly....usually...
That's why they pay me. To spot those things...
Also in this case the programmers -
none of them - fully understood assertions.