I agree that error handling can be improved but I don't think it should be part of this bug/change. I would like to keep it as small as possible. I'm fine with filing a separate bug for what you suggested and moving the discussion there. However, I would deal with IOException as suggested in the change if this is how it's currently done in EGit.