The XmppClient functions handleMUCMessage, handleMUCParticipantPresence, handleMUCSubject, handleMUCError receive a glooxwrapper::MUCRoom as the first argument.
That argument is currently unused.
If someone was to consider using it, he would get nothing but a null-deref segfault.
I suppose the intent of a library is not to segfault if one of its arguments is used.
Therefore either the argument must be removed because it is considered useless (from the glooxwrapper library POV),
or the argument must be provided, because it is considered useful (from the glooxwrapper library POV).
At some later time (needs more refactoring), one may become able to support using the XmppClient with multiple rooms.
That could have a realistic use case for current 0ad, since there is the players chat room and the moderators chat room.
But the point of the diff is to implement on the XmppClient what gloox provides us and pass it on to JS, so that JS can decide what to do with that information (Instead of segfaulting on top of it)