From 8e21030679a104fdb1ba3d02db6d3cb2920b55c1 Mon Sep 17 00:00:00 2001 From: Paul Stewart Date: Mon, 19 Dec 2016 08:45:32 -0800 Subject: [PATCH] DO NOT MERGE Fix other String.format issues in NetworkNotificationManager.java Commit fb2609d3eee1c7a4dda889c000f32183a044978a added a string format which caused an exception. On top of the fixes to this from e1406167186c2cabe683b3c1511ad6733e9106c9, we can also fix a few more issues here with some missing arguments to other String.format invocations in the aforementioned CL, and other formatting problems. Bug: 33737715 Test: Build and run SUW (cherry picked from commit 835cb49639fc7cb75819564fa15f2f5a0bd5dee2) Change-Id: I40064d3e1a0f0300c43bdbc51839b579a5394a36 --- .../android/server/connectivity/NetworkNotificationManager.java | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/services/core/java/com/android/server/connectivity/NetworkNotificationManager.java b/services/core/java/com/android/server/connectivity/NetworkNotificationManager.java index 777f7e7e0a7d..9166bb3d87f7 100644 --- a/services/core/java/com/android/server/connectivity/NetworkNotificationManager.java +++ b/services/core/java/com/android/server/connectivity/NetworkNotificationManager.java @@ -143,7 +143,8 @@ public class NetworkNotificationManager { if (DBG) { Slog.d(TAG, String.format( "showNotification tag=%s event=%s transport=%s extraInfo=%s highPrioriy=%s", - tag, nameOf(eventId), getTransportName(transportType), extraInfo, highPriority)); + tag, nameOf(eventId), getTransportName(transportType), extraInfo, + highPriority)); } Resources r = Resources.getSystem(); @@ -227,13 +228,14 @@ public class NetworkNotificationManager { } final int eventId = mNotificationTypeMap.get(id); if (DBG) { - Slog.d(TAG, String.format("clearing notification tag=%s event=", tag, nameOf(eventId))); + Slog.d(TAG, String.format("clearing notification tag=%s event=%s", tag, + nameOf(eventId))); } try { mNotificationManager.cancelAsUser(tag, eventId, UserHandle.ALL); } catch (NullPointerException npe) { Slog.d(TAG, String.format( - "failed to clear notification tag=%s event=", tag, nameOf(eventId)), npe); + "failed to clear notification tag=%s event=%s", tag, nameOf(eventId)), npe); } mNotificationTypeMap.delete(id); } -- 2.11.0