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

Commit 339d4477 authored by Hans Boehm's avatar Hans Boehm
Browse files

Add comment that mWrapped ==> mClosed 

This captures the discussion from ag/361549, which I found nonobvious.

Bug: 133424499
Test: Treehugger
Change-Id: Ia59aa3ef46e5e4e0dbb9411241feaae0b9919ad1
parent f376daf6

core/java/android/os/ParcelFileDescriptor.java

+2 −0
Original line number Diff line number Diff line
@@ -79,6 +79,7 @@ public class ParcelFileDescriptor implements Parcelable, Closeable { 
    /**

     * Wrapped {@link ParcelFileDescriptor}, if any. Used to avoid

     * double-closing {@link #mFd}.

     * mClosed is always true if mWrapped is non-null.

     */

    private final ParcelFileDescriptor mWrapped;
@@ -1018,6 +1019,7 @@ public class ParcelFileDescriptor implements Parcelable, Closeable { 
        }

        try {

            if (!mClosed) {

                // mWrapped was and is null.

                closeWithStatus(Status.LEAKED, null);

            }

        } finally {