summaryrefslogtreecommitdiff
path: root/baseLibrary
diff options
context:
space:
mode:
authorGeorge Mount <mount@google.com>2016-03-01 12:09:20 -0800
committerGeorge Mount <mount@google.com>2016-03-01 12:09:20 -0800
commit2bf1d3b17e04fd03ff447d57f770c89f935f52ae (patch)
tree1550a21d4dc47f1ab5bdf578d091eecd2571c0f6 /baseLibrary
parentb522c7650bf7d9ec566845bc9eb37e761eea853d (diff)
downloaddata-binding-2bf1d3b17e04fd03ff447d57f770c89f935f52ae.tar.gz
Check that callbacks are not null before adding them.
Fix future NullPointerException by checking for null callbacks when they are added. Change-Id: Icadd10d8f647a28cfe45977e18072cea17ea8a88
Diffstat (limited to 'baseLibrary')
-rw-r--r--baseLibrary/src/main/java/android/databinding/CallbackRegistry.java3
1 files changed, 3 insertions, 0 deletions
diff --git a/baseLibrary/src/main/java/android/databinding/CallbackRegistry.java b/baseLibrary/src/main/java/android/databinding/CallbackRegistry.java
index 0cd7b430..322b2794 100644
--- a/baseLibrary/src/main/java/android/databinding/CallbackRegistry.java
+++ b/baseLibrary/src/main/java/android/databinding/CallbackRegistry.java
@@ -210,6 +210,9 @@ public class CallbackRegistry<C, T, A> implements Cloneable {
* @param callback The callback to add.
*/
public synchronized void add(C callback) {
+ if (callback == null) {
+ throw new IllegalArgumentException("callback cannot be null");
+ }
int index = mCallbacks.lastIndexOf(callback);
if (index < 0 || isRemoved(index)) {
mCallbacks.add(callback);