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

Commit abfb1b0e authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Follow best practices for Atomic variables" into rvc-dev

parents 60143d44 98deae15
Loading
Loading
Loading
Loading
+11 −11
Original line number Diff line number Diff line
@@ -754,7 +754,7 @@ public class BugreportProgressService extends Service {
                != (info.lastProgress.intValue() / LOG_PROGRESS_STEP))) {
            Log.d(TAG, "Progress #" + info.id + ": " + percentageText);
        }
        info.lastProgress = new AtomicInteger(progress);
        info.lastProgress.set(progress);

        sendForegroundabledNotification(info.id, builder.build());
    }
@@ -1023,7 +1023,7 @@ public class BugreportProgressService extends Service {
        }
        Log.d(TAG, "Bugreport finished with title: " + info.getTitle()
                + " and shareDescription: " + info.shareDescription);
        info.finished = new AtomicBoolean(true);
        info.finished.set(true);

        synchronized (mLock) {
            // Stop running on foreground, otherwise share notification cannot be dismissed.
@@ -1807,18 +1807,18 @@ public class BugreportProgressService extends Service {
         * Current value of progress (in percentage) of the bugreport generation as
         * displayed by the UI.
         */
        AtomicInteger progress = new AtomicInteger(0);
        final AtomicInteger progress = new AtomicInteger(0);

        /**
         * Last value of progress (in percentage) of the bugreport generation for which
         * system notification was updated.
         */
        AtomicInteger lastProgress = new AtomicInteger(0);
        final AtomicInteger lastProgress = new AtomicInteger(0);

        /**
         * Time of the last progress update.
         */
        AtomicLong lastUpdate = new AtomicLong(System.currentTimeMillis());
        final AtomicLong lastUpdate = new AtomicLong(System.currentTimeMillis());

        /**
         * Time of the last progress update when Parcel was created.
@@ -1838,7 +1838,7 @@ public class BugreportProgressService extends Service {
        /**
         * Whether dumpstate sent an intent informing it has finished.
         */
        AtomicBoolean finished = new AtomicBoolean(false);
        final AtomicBoolean finished = new AtomicBoolean(false);

        /**
         * Whether the details entries have been added to the bugreport yet.
@@ -2073,8 +2073,8 @@ public class BugreportProgressService extends Service {
            initialName = in.readString();
            title = in.readString();
            description = in.readString();
            progress = new AtomicInteger(in.readInt());
            lastUpdate = new AtomicLong(in.readLong());
            progress.set(in.readInt());
            lastUpdate.set(in.readLong());
            formattedLastUpdate = in.readString();
            bugreportFile = readFile(in);

@@ -2083,7 +2083,7 @@ public class BugreportProgressService extends Service {
                  screenshotFiles.add(readFile(in));
            }

            finished = new AtomicBoolean(in.readInt() == 1);
            finished.set(in.readInt() == 1);
            screenshotCounter = in.readInt();
            shareDescription = in.readString();
            shareTitle = in.readString();
@@ -2155,8 +2155,8 @@ public class BugreportProgressService extends Service {
                        + ") from " + info.progress.intValue() + " to " + progress);
            }
        }
        info.progress = new AtomicInteger(progress);
        info.lastUpdate = new AtomicLong(System.currentTimeMillis());
        info.progress.set(progress);
        info.lastUpdate.set(System.currentTimeMillis());

        updateProgress(info);
    }