Actually seems like it should be a `const` function as it hardly changes the `QXmppClient`'s state, but since `socketError()` also isn't `const` it's not a really big deal. If you don't feel like changing it I'll happily merge this PR.