I'm not sure how many places in our code know that error codes greater than zero indicate failure, but it seems to me that it would aid readability and just possibly maintainability to make a static method, maybe on IcuWrappers, to encapsulate this.

If we're already doing this widely and the change would make the change set much bigger it's probably not worth it (at least as part of this one).