[jitsi-dev] [Patch] null "chatrooms" for null chat provider


#1

In the case where the provided chatRoomProvider == null simply return
null. If not, we cause an NPE.

This may (also) be fixed at the root, i.e. check that we do not pass
null, however we already have to check for null since there is another
case that potentially returns null, so we're "attaching" to already
defined behavior.

no-chatrooms-for-null-chatroomprovider.patch (637 Bytes)

···

--
Danny


#2

Hello Danny,

Thanks for the patch! It is now committed and ack-ed.

···

On Tue, Dec 3, 2013 at 8:00 PM, Danny van Heumen <danny@dannyvanheumen.nl>wrote:

In the case where the provided chatRoomProvider == null simply return
null. If not, we cause an NPE.

This may (also) be fixed at the root, i.e. check that we do not pass
null, however we already have to check for null since there is another
case that potentially returns null, so we're "attaching" to already
defined behavior.

--
Danny

_______________________________________________
dev mailing list
dev@jitsi.org
Unsubscribe instructions and other list options:
http://lists.jitsi.org/mailman/listinfo/dev

--
Regards,
Hristo.