Why is that bad? It’s no different conceptually than:
void TrySomething() {
try {
} catch (ArgumentException) {
HandleTrySomethingFailure();
}
}
void HandleTrySomethingFailure() {
try {
} catch (IndexOutOfRangeException) {
}
}
Before you go over there and give him a piece of your brain (try the parietal lobe, it’s particularly offensive) , what exactly are you going to say to him? How will you answer the proverbial “why?”
What’s even more ironic is that when the jitter inlines this code, it will look exactly like your example.
-Oisin