Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 45f533c1 authored by Andrew Scull's avatar Andrew Scull
Browse files

NetworkManagementService.java: fix error-prone warnings

- Lock objects should be final
- Inner classes should be static if possible

Change-Id: Idca268e03ea1e9f80e64012d1e011dc8bca1d9a8
Merged-In: Idca268e03ea1e9f80e64012d1e011dc8bca1d9a8
Fix: 38446237
Test: Build
parent a2663b00
Loading
Loading
Loading
Loading
+3 −3
Original line number Diff line number Diff line
@@ -149,7 +149,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub
     */
    public static final String PERMISSION_SYSTEM = "SYSTEM";

    class NetdResponseCode {
    static class NetdResponseCode {
        /* Keep in sync with system/netd/server/ResponseCode.h */
        public static final int InterfaceListResult       = 110;
        public static final int TetherInterfaceListResult = 111;
@@ -220,7 +220,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub

    private final NetworkStatsFactory mStatsFactory = new NetworkStatsFactory();

    private Object mQuotaLock = new Object();
    private final Object mQuotaLock = new Object();

    /** Set of interfaces with active quotas. */
    @GuardedBy("mQuotaLock")
@@ -265,7 +265,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub
    @GuardedBy("mQuotaLock")
    private boolean mDataSaverMode;

    private Object mIdleTimerLock = new Object();
    private final Object mIdleTimerLock = new Object();
    /** Set of interfaces with active idle timers. */
    private static class IdleTimerParams {
        public final int timeout;