From 017ee9e034bcb106b71c16d905b9f0982b01f433 Mon Sep 17 00:00:00 2001 From: ardevd Date: Wed, 17 Apr 2019 23:50:18 +0200 Subject: [PATCH] controllers: removed identical sub-expression in if statement. (#529) Using the same value on either side of a binary operator is almost always a mistake. Either there is supposed to be a null check for something else in there or the second sub expression should be removed. Signed-off-by: eho --- .../nextcloud/talk/controllers/CallNotificationController.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/java/com/nextcloud/talk/controllers/CallNotificationController.java b/app/src/main/java/com/nextcloud/talk/controllers/CallNotificationController.java index d1abe243f..435eb41f1 100644 --- a/app/src/main/java/com/nextcloud/talk/controllers/CallNotificationController.java +++ b/app/src/main/java/com/nextcloud/talk/controllers/CallNotificationController.java @@ -445,7 +445,7 @@ public class CallNotificationController extends BaseController { script.forEach(output); output.copyTo(resource); - if (backgroundImageView != null && backgroundImageView != null) { + if (backgroundImageView != null) { backgroundImageView.setImageDrawable(new BitmapDrawable(resource)); } } else if (AvatarStatusCodeHolder.getInstance().getStatusCode() == 201) {