aboutsummaryrefslogtreecommitdiff
path: root/juick-notifications/src/main/java/com/juick/components/Notifications.java
diff options
context:
space:
mode:
authorGravatar Alexander Alexeev2016-11-30 15:10:15 +0700
committerGravatar Alexander Alexeev2016-11-30 15:10:15 +0700
commit1d837a835c04078ec66325e9fb9de21ac401874e (patch)
tree428217d2c617b9caf2eb6d42a7716789881d0937 /juick-notifications/src/main/java/com/juick/components/Notifications.java
parentb45c000ed478f2407288aab1532425288f998689 (diff)
prevent string concatenation for disabled loggers
Diffstat (limited to 'juick-notifications/src/main/java/com/juick/components/Notifications.java')
-rw-r--r--juick-notifications/src/main/java/com/juick/components/Notifications.java8
1 files changed, 4 insertions, 4 deletions
diff --git a/juick-notifications/src/main/java/com/juick/components/Notifications.java b/juick-notifications/src/main/java/com/juick/components/Notifications.java
index dac4d22e..f98fdcce 100644
--- a/juick-notifications/src/main/java/com/juick/components/Notifications.java
+++ b/juick-notifications/src/main/java/com/juick/components/Notifications.java
@@ -133,12 +133,12 @@ public class Notifications implements AutoCloseable {
MulticastResult result = GCMSender.send(message, regids, 3);
List<Result> results = result.getResults();
for (int i = 0; i < results.size(); i++) {
- logger.info("RES " + i + ": " + results.get(i).toString());
+ logger.info("RES {}: {}", i, results.get(i));
}
} catch (IOException ex) {
logger.error(ex.getMessage(), ex);
} catch (IllegalArgumentException err) {
- logger.warn("Android: Invalid API Key");
+ logger.warn("Android: Invalid API Key", err);
}
} else {
logger.info("GMS: no recipients");
@@ -183,7 +183,7 @@ public class Notifications implements AutoCloseable {
+ "</toast>";
logger.trace(xml);
for (String url : urls) {
- logger.info("WNS: " + url);
+ logger.info("WNS: {}", url);
sendWNS(wnsToken, url, xml);
}
} catch (IOException | IllegalStateException ex) {
@@ -207,7 +207,7 @@ public class Notifications implements AutoCloseable {
if (!tokens.isEmpty()) {
for (String token : tokens) {
String payload = APNS.newPayload().alertTitle("@" + jmsg.getUser().getName()).alertBody(jmsg.getText()).build();
- logger.info("APNS: " + token);
+ logger.info("APNS: {}", token);
apns.push(token, payload);
}
} else {