Merge pull request #179 from damencho/4.2
Makes xmpperror descriptive text optional as said in the rfc.
Florian Schmaus <flo@geekplace.eu>
a66c42834fe93c6a0fb221be3aee43f5278aa50a
Add findServicesDiscoveryInfo() variant
which does not log some exceptions, but instead comes with an optional
output paramater which returns the encountered exceptions.
Florian Schmaus <flo@geekplace.eu>
5ef6853db6873115d53d9bb65780b6f5a8d549f9
Improve MultiUserChat's leave() and destroy() login
Remove the "if (!joined) return" guard in leave() this allows to
resync the instances state with the real world state in case they ever
get out of sync.
Also call userHasLeft() in even if leave() throws and in certain
situations if destroy() throws.
Thanks to Дамян Минков and Ingo Bauersachs for pointing this out.
Florian Schmaus <flo@geekplace.eu>
d93d214f0eda649be34d6cc3f616ead74d46ee49
Remove duplicate code in MultiUserChat
Florian Schmaus <flo@geekplace.eu>
0a4cd79d4ea0111dbe7e25e8ca191a2c40b53bed
Call MultiUserChat.userHasLeft() before invoking listeners
in case a listener throws, userHasLeft() will eventually have set the
MutliUserChat instance to the right state.