Skip to content
Snippets Groups Projects
Commit 48165939 authored by Christopher Ferris's avatar Christopher Ferris Committed by Gerrit Code Review
Browse files

Merge "Use a file descriptor in decodeFile." into main

parents cb1487b4 39d091bc
No related branches found
No related tags found
No related merge requests found
......@@ -25,10 +25,13 @@ import android.content.res.AssetManager;
import android.content.res.Resources;
import android.os.Build;
import android.os.Trace;
import android.system.OsConstants;
import android.util.DisplayMetrics;
import android.util.Log;
import android.util.TypedValue;
import libcore.io.IoBridge;
import java.io.FileDescriptor;
import java.io.FileInputStream;
import java.io.IOException;
......@@ -523,19 +526,19 @@ public class BitmapFactory {
public static Bitmap decodeFile(String pathName, Options opts) {
validate(opts);
Bitmap bm = null;
InputStream stream = null;
FileDescriptor fd = null;
try {
stream = new FileInputStream(pathName);
bm = decodeStream(stream, null, opts);
fd = IoBridge.open(pathName, OsConstants.O_RDONLY);
bm = decodeFileDescriptor(fd, null, opts);
} catch (Exception e) {
/* do nothing.
If the exception happened on open, bm will be null.
*/
Log.e("BitmapFactory", "Unable to decode stream: " + e);
Log.e("BitmapFactory", "Unable to decode file: " + e);
} finally {
if (stream != null) {
if (fd != null) {
try {
stream.close();
IoBridge.closeAndSignalBlockedThreads(fd);
} catch (IOException e) {
// do nothing here
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment