null check for response listener

mcxiaoke · mcxiaoke · commit 798ce09c4bdf · 2015-09-08T13:21:01.000+08:00
diff --git a/src/main/java/com/android/volley/toolbox/JsonRequest.java b/src/main/java/com/android/volley/toolbox/JsonRequest.java
@@ -68,7 +68,9 @@ protected void onFinish() {
 
     @Override
     protected void deliverResponse(T response) {
-        mListener.onResponse(response);
+        if (mListener != null) {
+            mListener.onResponse(response);
+        }
     }
 
     @Override
diff --git a/src/main/java/com/android/volley/toolbox/StringRequest.java b/src/main/java/com/android/volley/toolbox/StringRequest.java
@@ -63,7 +63,9 @@ protected void onFinish() {
 
     @Override
     protected void deliverResponse(String response) {
-        mListener.onResponse(response);
+        if (mListener != null) {
+            mListener.onResponse(response);
+        }
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,9 @@ protected void onFinish() {`
`68`	`68`
`69`	`69`	`@Override`
`70`	`70`	`protected void deliverResponse(T response) {`
`71`		`- mListener.onResponse(response);`
	`71`	`+ if (mListener != null) {`
	`72`	`+ mListener.onResponse(response);`
	`73`	`+ }`
`72`	`74`	`}`
`73`	`75`
`74`	`76`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,9 @@ protected void onFinish() {`
`63`	`63`
`64`	`64`	`@Override`
`65`	`65`	`protected void deliverResponse(String response) {`
`66`		`- mListener.onResponse(response);`
	`66`	`+ if (mListener != null) {`
	`67`	`+ mListener.onResponse(response);`
	`68`	`+ }`
`67`	`69`	`}`
`68`	`70`
`69`	`71`	`@Override`