LineageOS/android_packages_modules_Connectivity
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix other String.format issues in NetworkNotificationManager.java

Browse Source 
Commit fd35080555 added a string
format which caused an exception.  On top of the fixes to this
from 75017f83c7, 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
Change-Id: I13410ffebbc606b8801656ea0934f96ccd4b5357
This commit is contained in:
Paul Stewart
2016-12-19 08:45:32 -08:00
parent 384daf99a9
commit c135a35fca
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
services/core/java/com/android/server/connectivity/NetworkNotificationManager.java
View File

@@ -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);
}