Issue 344-related -- avoid NPE when previewFormatString is null

git-svn-id: https://zxing.googlecode.com/svn/trunk@1205 59b500cc-1b3d-0410-9834-0bbf25fbcc57
This commit is contained in:
srowen 2010-02-09 09:22:33 +00:00
parent 49601e62e5
commit 146c4e096f

View file

@ -287,8 +287,6 @@ final class CameraManager {
// This is the standard Android format which all devices are REQUIRED to support.
// In theory, it's the only one we should ever care about.
case PixelFormat.YCbCr_420_SP:
return new PlanarYUVLuminanceSource(data, width, height, rect.left, rect.top,
rect.width(), rect.height());
// This format has never been seen in the wild, but is compatible as we only care
// about the Y channel, so allow it.
case PixelFormat.YCbCr_422_SP:
@ -297,7 +295,7 @@ final class CameraManager {
default:
// The Samsung Moment incorrectly uses this variant instead of the 'sp' version.
// Fortunately, it too has all the Y data up front, so we can read it.
if (previewFormatString.equals("yuv420p")) {
if ("yuv420p".equals(previewFormatString)) {
return new PlanarYUVLuminanceSource(data, width, height, rect.left, rect.top,
rect.width(), rect.height());
}