This does look a little clearer, but it changes the functionality slightly. Before it was calling super before checking for the error section, now it checks it regardless of whether super returns something. I actually don't think it makes a difference though because reading xml isn't going to be much of a performance hit. Up to you if you want to change it or not.