From 5cc1df3c92bf00ea5cb5f24198d119cca6a78459 Mon Sep 17 00:00:00 2001 From: dswitkin Date: Sun, 13 Dec 2009 21:08:57 +0000 Subject: [PATCH] Finished work on the local binarizer and renamed it to HybridBinarizer. It uses the old histogram for 1D and the new approach for 2D. The unit tests don't fully demonstrate how much better this is in real-world lighting, but it pretty much solves the situation of pointing at a QR Code for 30 seconds without scanning, due to a shadow or gradient. HybridBinarizer is now the default across all the fast clients and the unit tests. git-svn-id: https://zxing.googlecode.com/svn/trunk@1157 59b500cc-1b3d-0410-9834-0bbf25fbcc57 --- .../zxing/client/android/DecodeThread.java | 4 +- .../client/androidtest/BenchmarkThread.java | 4 +- .../common/GlobalHistogramBinarizer.java | 6 +- ...ockBinarizer.java => HybridBinarizer.java} | 74 +++++++----------- core/test/data/blackbox/pdf417/01.png | Bin 842 -> 856 bytes .../common/AbstractBlackBoxTestCase.java | 4 +- .../AbstractNegativeBlackBoxTestCase.java | 2 +- .../DataMatrixBlackBox2TestCase.java | 6 +- .../zxing/pdf417/PDF417BlackBox2TestCase.java | 4 +- .../zxing/qrcode/QRCodeBlackBox1TestCase.java | 10 +-- .../zxing/qrcode/QRCodeBlackBox2TestCase.java | 6 +- .../zxing/qrcode/QRCodeBlackBox3TestCase.java | 10 +-- .../zxing/qrcode/QRCodeBlackBox4TestCase.java | 8 +- .../zxing/qrcode/QRCodeBlackBox5TestCase.java | 9 ++- .../zxing/client/j2se/CommandLineRunner.java | 6 +- .../google/zxing/client/j2se/GUIRunner.java | 4 +- .../zxing/client/j2se/ImageConverter.java | 4 +- .../com/google/zxing/web/DecodeEmailTask.java | 6 +- .../com/google/zxing/web/DecodeServlet.java | 8 +- 19 files changed, 81 insertions(+), 94 deletions(-) rename core/src/com/google/zxing/common/{LocalBlockBinarizer.java => HybridBinarizer.java} (72%) diff --git a/android/src/com/google/zxing/client/android/DecodeThread.java b/android/src/com/google/zxing/client/android/DecodeThread.java index ca2afea8b..b383b4760 100755 --- a/android/src/com/google/zxing/client/android/DecodeThread.java +++ b/android/src/com/google/zxing/client/android/DecodeThread.java @@ -23,7 +23,7 @@ import com.google.zxing.MultiFormatReader; import com.google.zxing.ReaderException; import com.google.zxing.Result; import com.google.zxing.ResultPointCallback; -import com.google.zxing.common.GlobalHistogramBinarizer; +import com.google.zxing.common.HybridBinarizer; import android.content.SharedPreferences; import android.os.Bundle; @@ -165,7 +165,7 @@ final class DecodeThread extends Thread { long start = System.currentTimeMillis(); Result rawResult = null; PlanarYUVLuminanceSource source = CameraManager.get().buildLuminanceSource(data, width, height); - BinaryBitmap bitmap = new BinaryBitmap(new GlobalHistogramBinarizer(source)); + BinaryBitmap bitmap = new BinaryBitmap(new HybridBinarizer(source)); try { rawResult = multiFormatReader.decodeWithState(bitmap); } catch (ReaderException re) { diff --git a/androidtest/src/com/google/zxing/client/androidtest/BenchmarkThread.java b/androidtest/src/com/google/zxing/client/androidtest/BenchmarkThread.java index 2f320163a..eaeddfe00 100755 --- a/androidtest/src/com/google/zxing/client/androidtest/BenchmarkThread.java +++ b/androidtest/src/com/google/zxing/client/androidtest/BenchmarkThread.java @@ -20,7 +20,7 @@ import com.google.zxing.BinaryBitmap; import com.google.zxing.MultiFormatReader; import com.google.zxing.ReaderException; import com.google.zxing.Result; -import com.google.zxing.common.GlobalHistogramBinarizer; +import com.google.zxing.common.HybridBinarizer; import android.os.Debug; import android.os.Message; @@ -94,7 +94,7 @@ final class BenchmarkThread extends Thread { // scheduling and what else is happening in the system. long now = Debug.threadCpuTimeNanos(); try { - BinaryBitmap bitmap = new BinaryBitmap(new GlobalHistogramBinarizer(source)); + BinaryBitmap bitmap = new BinaryBitmap(new HybridBinarizer(source)); result = mMultiFormatReader.decodeWithState(bitmap); success = true; } catch (ReaderException e) { diff --git a/core/src/com/google/zxing/common/GlobalHistogramBinarizer.java b/core/src/com/google/zxing/common/GlobalHistogramBinarizer.java index f86525e65..015dfa939 100644 --- a/core/src/com/google/zxing/common/GlobalHistogramBinarizer.java +++ b/core/src/com/google/zxing/common/GlobalHistogramBinarizer.java @@ -26,10 +26,12 @@ import com.google.zxing.ReaderException; * algorithm. However, because it picks a global black point, it cannot handle difficult shadows * and gradients. * + * Faster mobile devices and all desktop applications should probably use HybridBinarizer instead. + * * @author dswitkin@google.com (Daniel Switkin) * @author Sean Owen */ -public final class GlobalHistogramBinarizer extends Binarizer { +public class GlobalHistogramBinarizer extends Binarizer { private static final int LUMINANCE_BITS = 5; private static final int LUMINANCE_SHIFT = 8 - LUMINANCE_BITS; @@ -105,7 +107,7 @@ public final class GlobalHistogramBinarizer extends Binarizer { for (int y = 0; y < height; y++) { int offset = y * width; for (int x = 0; x< width; x++) { - int pixel = localLuminances[offset + x] & 0xff; + int pixel = localLuminances[offset + x] & 0xff; if (pixel < blackPoint) { matrix.set(x, y); } diff --git a/core/src/com/google/zxing/common/LocalBlockBinarizer.java b/core/src/com/google/zxing/common/HybridBinarizer.java similarity index 72% rename from core/src/com/google/zxing/common/LocalBlockBinarizer.java rename to core/src/com/google/zxing/common/HybridBinarizer.java index 8f3709827..06eb729c3 100644 --- a/core/src/com/google/zxing/common/LocalBlockBinarizer.java +++ b/core/src/com/google/zxing/common/HybridBinarizer.java @@ -18,6 +18,7 @@ package com.google.zxing.common; import com.google.zxing.Binarizer; import com.google.zxing.LuminanceSource; +import com.google.zxing.ReaderException; /** * This class implements a local thresholding algorithm, which while slower than the @@ -27,51 +28,56 @@ import com.google.zxing.LuminanceSource; * However it tends to produce artifacts on lower frequency images and is therefore not * a good general purpose binarizer for uses outside ZXing. * - * NOTE: This class is still experimental and may not be ready for prime time yet. + * This class extends GlobalHistogramBinarizer, using the older histogram approach for 1D readers, + * and the newer local approach for 2D readers. 1D decoding using a per-row histogram is already + * inherently local, and only fails for horizontal gradients. We can revisit that problem later, + * but for now it was not a win to use local blocks for 1D. + * + * This Binarizer is the default for the unit tests and the recommended class for library users. * * @author dswitkin@google.com (Daniel Switkin) */ -public final class LocalBlockBinarizer extends Binarizer { +public final class HybridBinarizer extends GlobalHistogramBinarizer { + + // This class uses 5x5 blocks to compute local luminance, where each block is 8x8 pixels. + // So this is the smallest dimension in each axis we can accept. + private static final int MINIMUM_DIMENSION = 40; private BitMatrix matrix = null; - public LocalBlockBinarizer(LuminanceSource source) { + public HybridBinarizer(LuminanceSource source) { super(source); } - // TODO: Consider a different strategy for 1D Readers. - public BitArray getBlackRow(int y, BitArray row) { - binarizeEntireImage(); - return matrix.getRow(y, row); - } - - // TODO: If getBlackRow() calculates its own values, removing sharpening here. - public BitMatrix getBlackMatrix() { + public BitMatrix getBlackMatrix() throws ReaderException { binarizeEntireImage(); return matrix; } public Binarizer createBinarizer(LuminanceSource source) { - return new LocalBlockBinarizer(source); + return new HybridBinarizer(source); } // Calculates the final BitMatrix once for all requests. This could be called once from the // constructor instead, but there are some advantages to doing it lazily, such as making // profiling easier, and not doing heavy lifting when callers don't expect it. - private void binarizeEntireImage() { + private void binarizeEntireImage() throws ReaderException { if (matrix == null) { LuminanceSource source = getLuminanceSource(); - byte[] luminances = source.getMatrix(); - int width = source.getWidth(); - int height = source.getHeight(); - sharpenRow(luminances, width, height); + if (source.getWidth() >= MINIMUM_DIMENSION && source.getHeight() >= MINIMUM_DIMENSION) { + byte[] luminances = source.getMatrix(); + int width = source.getWidth(); + int height = source.getHeight(); + int subWidth = width >> 3; + int subHeight = height >> 3; + int[][] blackPoints = calculateBlackPoints(luminances, subWidth, subHeight, width); - int subWidth = width >> 3; - int subHeight = height >> 3; - int[][] blackPoints = calculateBlackPoints(luminances, subWidth, subHeight, width); - - matrix = new BitMatrix(width, height); - calculateThresholdForBlock(luminances, subWidth, subHeight, width, blackPoints, matrix); + matrix = new BitMatrix(width, height); + calculateThresholdForBlock(luminances, subWidth, subHeight, width, blackPoints, matrix); + } else { + // If the image is too small, fall back to the global histogram approach. + matrix = super.getBlackMatrix(); + } } } @@ -148,26 +154,4 @@ public final class LocalBlockBinarizer extends Binarizer { return blackPoints; } - // Applies a simple -1 4 -1 box filter with a weight of 2 to each row. - private static void sharpenRow(byte[] luminances, int width, int height) { - for (int y = 0; y < height; y++) { - int offset = y * width; - int left = luminances[offset] & 0xff; - int center = luminances[offset + 1] & 0xff; - for (int x = 1; x < width - 1; x++) { - int right = luminances[offset + x + 1] & 0xff; - int pixel = ((center << 2) - left - right) >> 1; - // Must clamp values to 0..255 so they will fit in a byte. - if (pixel > 255) { - pixel = 255; - } else if (pixel < 0) { - pixel = 0; - } - luminances[offset + x] = (byte)pixel; - left = center; - center = right; - } - } - } - } diff --git a/core/test/data/blackbox/pdf417/01.png b/core/test/data/blackbox/pdf417/01.png index 6ac92598e898e0fd2c2eed463b1f0ed6d003db45..48549237f6d4c9c056631c3af72d659dff95001f 100644 GIT binary patch delta 801 zcmV++1K#|~2G|CW7Ya}a00000S3^Ypks%fZ|A&vvzmYs2e+&&291_5u^&$WO0^CVN zK~#9!?OWS+gdhxLIR5`1*Xe_@T!bnR(6#ZLw!2Y;Ny3%b`KdC<}P%$(ZPtY}{*&tbsm{)$mKfJvg?!zJbD+FOB;cTm{m7I$W_y z8F4C-@hRt`e$$V$9AEC1d^0$FH#p2OCfYP*Cp#XRf2V0s%ThH3B0VyS42>}c_yLSD zWVj$6z|oD}b8{q2>Stk(CWZ33U9y!89y|Z(Vs9qXxpkKY^@oukktUy8+z7hLWW0TtggtB_4>MHoQLO_i~*Cxyk`03SYZPh=OqdqMWDx=HKBmhv*8}^& zv2k+D@xVTCC``?hrBT#$FcaN!wB1tEaRj*Ue9e2QvQYeXW3HQRGEpFodU8#ksLQkB zGsiy3+UEevJvzn7hJuK{SjnZnGW_D3!97(UT?HbvN0Bos@r{{yT4y|VRvm-#z^8Eq zf0CJZ-1+GO`MWeRjFoXL{HbV z^2QJnj?8b>J9^^kmJ04xaJ`S=i{r_@^Igg6=#WCVYJp3-ApZU?9)Una4*e&11*hN? foPzuB;C=zpHVnq65sm5G?oAW0?tW9 zK~!ko?OWNBq#zI^3IG3(*W3rCIfPM)0*}CmndvS?s4J<10nK+W`(fV9vj0Uwr$O-J zpIEWcRyc}RJ(96z|Lv%THhP)_v3+us!#hGsZlz2{Gk9OM=1>qu)nNY}&ofKP`MLZa z-o+f0PCqM|m$!;iAUCS$j|jOJm4 zeq&n<2QU1c(MA0~OXgCGXEdRX`L@$6{Lu`FAfYd+I8LbEMifC1~i z11*8me?{O{C~7fF^1>q2BTo+Wxn8@Qme%X*mT?}QLox|{` z*505ok~y4^wKwRC!m7>05k-2;oSR{kS)))}e_(>NC?<;taQ87?p0O6##vFx{V~hp1 zF^9s`JXso5RR=TBEPLxMRULaU_s%1yOO=M=(~Y@qw#h_+ILgU2d7>`Qir*ajBx{=i zEceJ1Ck_P_d$F2J{bcyXFEjU4c{CM>AdezPRN{r1cv@#Xc2*sN@_?st1(KOF?)-Iu zbo^UH2xD~&3xA4T-3Gg-JSZ(6{n3=q29#N5n9M0=;;x^KY+jt61mYpqL~>#X2}b5! z^^TtSG|S1{t<1GT|EMU3LWzwa$kD=2T>>tv~-3~ QDF6Tf07*qoM6N<$f(@!`MF0Q* diff --git a/core/test/src/com/google/zxing/common/AbstractBlackBoxTestCase.java b/core/test/src/com/google/zxing/common/AbstractBlackBoxTestCase.java index dd95570fd..2ef12b43d 100644 --- a/core/test/src/com/google/zxing/common/AbstractBlackBoxTestCase.java +++ b/core/test/src/com/google/zxing/common/AbstractBlackBoxTestCase.java @@ -184,7 +184,7 @@ public abstract class AbstractBlackBoxTestCase extends TestCase { float rotation = testResults.get(x).getRotation(); BufferedImage rotatedImage = rotateImage(image, rotation); LuminanceSource source = new BufferedImageLuminanceSource(rotatedImage); - BinaryBitmap bitmap = new BinaryBitmap(new GlobalHistogramBinarizer(source)); + BinaryBitmap bitmap = new BinaryBitmap(new HybridBinarizer(source)); if (decode(bitmap, rotation, expectedText, false)) { passedCounts[x]++; } @@ -294,4 +294,4 @@ public abstract class AbstractBlackBoxTestCase extends TestCase { } } -} \ No newline at end of file +} diff --git a/core/test/src/com/google/zxing/common/AbstractNegativeBlackBoxTestCase.java b/core/test/src/com/google/zxing/common/AbstractNegativeBlackBoxTestCase.java index be67eb171..88f82497d 100644 --- a/core/test/src/com/google/zxing/common/AbstractNegativeBlackBoxTestCase.java +++ b/core/test/src/com/google/zxing/common/AbstractNegativeBlackBoxTestCase.java @@ -109,7 +109,7 @@ public abstract class AbstractNegativeBlackBoxTestCase extends AbstractBlackBoxT private boolean checkForFalsePositives(BufferedImage image, float rotationInDegrees) { BufferedImage rotatedImage = rotateImage(image, rotationInDegrees); LuminanceSource source = new BufferedImageLuminanceSource(rotatedImage); - BinaryBitmap bitmap = new BinaryBitmap(new GlobalHistogramBinarizer(source)); + BinaryBitmap bitmap = new BinaryBitmap(new HybridBinarizer(source)); Result result; try { result = getReader().decode(bitmap); diff --git a/core/test/src/com/google/zxing/datamatrix/DataMatrixBlackBox2TestCase.java b/core/test/src/com/google/zxing/datamatrix/DataMatrixBlackBox2TestCase.java index 710bbe7e3..3508ff0db 100644 --- a/core/test/src/com/google/zxing/datamatrix/DataMatrixBlackBox2TestCase.java +++ b/core/test/src/com/google/zxing/datamatrix/DataMatrixBlackBox2TestCase.java @@ -27,10 +27,10 @@ public final class DataMatrixBlackBox2TestCase extends AbstractBlackBoxTestCase public DataMatrixBlackBox2TestCase() { // TODO use MultiFormatReader here once Data Matrix decoder is done super("test/data/blackbox/datamatrix-2", new DataMatrixReader(), BarcodeFormat.DATAMATRIX); - addTest(2, 2, 0.0f); + addTest(4, 4, 0.0f); addTest(1, 1, 90.0f); - addTest(4, 4, 180.0f); - addTest(3, 3, 270.0f); + addTest(3, 3, 180.0f); + addTest(1, 1, 270.0f); } } diff --git a/core/test/src/com/google/zxing/pdf417/PDF417BlackBox2TestCase.java b/core/test/src/com/google/zxing/pdf417/PDF417BlackBox2TestCase.java index 06c28e7f9..a1702aed2 100644 --- a/core/test/src/com/google/zxing/pdf417/PDF417BlackBox2TestCase.java +++ b/core/test/src/com/google/zxing/pdf417/PDF417BlackBox2TestCase.java @@ -34,7 +34,7 @@ public final class PDF417BlackBox2TestCase extends AbstractBlackBoxTestCase { public PDF417BlackBox2TestCase() { super("test/data/blackbox/pdf417-2", new MultiFormatReader(), BarcodeFormat.PDF417); addTest(11, 11, 0.0f); - addTest(12, 12, 180.0f); + addTest(11, 11, 180.0f); } @Override @@ -46,4 +46,4 @@ public final class PDF417BlackBox2TestCase extends AbstractBlackBoxTestCase { return table; } -} \ No newline at end of file +} diff --git a/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox1TestCase.java b/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox1TestCase.java index 544627c43..b0e164586 100644 --- a/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox1TestCase.java +++ b/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox1TestCase.java @@ -27,10 +27,10 @@ public final class QRCodeBlackBox1TestCase extends AbstractBlackBoxTestCase { public QRCodeBlackBox1TestCase() { super("test/data/blackbox/qrcode-1", new MultiFormatReader(), BarcodeFormat.QR_CODE); - addTest(19, 19, 0.0f); - addTest(15, 15, 90.0f); - addTest(17, 17, 180.0f); - addTest(15, 15, 270.0f); + addTest(17, 17, 0.0f); + addTest(13, 13, 90.0f); + addTest(16, 16, 180.0f); + addTest(14, 14, 270.0f); } -} \ No newline at end of file +} diff --git a/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox2TestCase.java b/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox2TestCase.java index 658b3238b..a5bac913b 100644 --- a/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox2TestCase.java +++ b/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox2TestCase.java @@ -27,10 +27,10 @@ public final class QRCodeBlackBox2TestCase extends AbstractBlackBoxTestCase { public QRCodeBlackBox2TestCase() { super("test/data/blackbox/qrcode-2", new MultiFormatReader(), BarcodeFormat.QR_CODE); - addTest(23, 23, 0.0f); - addTest(21, 21, 90.0f); + addTest(21, 21, 0.0f); + addTest(18, 18, 90.0f); addTest(20, 20, 180.0f); addTest(18, 18, 270.0f); } -} \ No newline at end of file +} diff --git a/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox3TestCase.java b/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox3TestCase.java index 3f000a537..8b152960b 100644 --- a/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox3TestCase.java +++ b/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox3TestCase.java @@ -27,10 +27,10 @@ public final class QRCodeBlackBox3TestCase extends AbstractBlackBoxTestCase { public QRCodeBlackBox3TestCase() { super("test/data/blackbox/qrcode-3", new MultiFormatReader(), BarcodeFormat.QR_CODE); - addTest(33, 33, 0.0f); - addTest(36, 36, 90.0f); - addTest(32, 32, 180.0f); - addTest(38, 38, 270.0f); + addTest(36, 36, 0.0f); + addTest(37, 37, 90.0f); + addTest(35, 35, 180.0f); + addTest(37, 37, 270.0f); } -} \ No newline at end of file +} diff --git a/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox4TestCase.java b/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox4TestCase.java index bd3d06431..49db0c922 100644 --- a/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox4TestCase.java +++ b/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox4TestCase.java @@ -29,10 +29,10 @@ public final class QRCodeBlackBox4TestCase extends AbstractBlackBoxTestCase { public QRCodeBlackBox4TestCase() { super("test/data/blackbox/qrcode-4", new MultiFormatReader(), BarcodeFormat.QR_CODE); - addTest(32, 32, 0.0f); - addTest(37, 37, 90.0f); - addTest(36, 36, 180.0f); + addTest(35, 35, 0.0f); + addTest(36, 36, 90.0f); + addTest(34, 34, 180.0f); addTest(34, 34, 270.0f); } -} \ No newline at end of file +} diff --git a/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox5TestCase.java b/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox5TestCase.java index 07db2ccc3..8fca08ef8 100644 --- a/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox5TestCase.java +++ b/core/test/src/com/google/zxing/qrcode/QRCodeBlackBox5TestCase.java @@ -23,6 +23,7 @@ import com.google.zxing.common.AbstractBlackBoxTestCase; /** * Some very difficult exposure conditions including self-shadowing, which happens a lot when * pointing down at a barcode (i.e. the phone's shadow falls across part of the image). + * The global histogram gets about 5/15, where the local one gets 15/15. * * @author dswitkin@google.com (Daniel Switkin) */ @@ -30,10 +31,10 @@ public final class QRCodeBlackBox5TestCase extends AbstractBlackBoxTestCase { public QRCodeBlackBox5TestCase() { super("test/data/blackbox/qrcode-5", new MultiFormatReader(), BarcodeFormat.QR_CODE); - addTest(5, 5, 0.0f); - addTest(4, 4, 90.0f); - addTest(5, 5, 180.0f); - addTest(5, 5, 270.0f); + addTest(15, 15, 0.0f); + addTest(15, 15, 90.0f); + addTest(15, 15, 180.0f); + addTest(15, 15, 270.0f); } } diff --git a/javase/src/com/google/zxing/client/j2se/CommandLineRunner.java b/javase/src/com/google/zxing/client/j2se/CommandLineRunner.java index 92c8194d5..a7ad9d62a 100644 --- a/javase/src/com/google/zxing/client/j2se/CommandLineRunner.java +++ b/javase/src/com/google/zxing/client/j2se/CommandLineRunner.java @@ -16,18 +16,18 @@ package com.google.zxing.client.j2se; +import com.google.zxing.BarcodeFormat; import com.google.zxing.BinaryBitmap; import com.google.zxing.DecodeHintType; import com.google.zxing.LuminanceSource; import com.google.zxing.MultiFormatReader; import com.google.zxing.ReaderException; import com.google.zxing.Result; -import com.google.zxing.BarcodeFormat; import com.google.zxing.client.result.ParsedResult; import com.google.zxing.client.result.ResultParser; import com.google.zxing.common.BitArray; import com.google.zxing.common.BitMatrix; -import com.google.zxing.common.GlobalHistogramBinarizer; +import com.google.zxing.common.HybridBinarizer; import java.awt.image.BufferedImage; import java.io.File; @@ -200,7 +200,7 @@ public final class CommandLineRunner { } try { LuminanceSource source = new BufferedImageLuminanceSource(image); - BinaryBitmap bitmap = new BinaryBitmap(new GlobalHistogramBinarizer(source)); + BinaryBitmap bitmap = new BinaryBitmap(new HybridBinarizer(source)); if (dumpBlackPoint) { dumpBlackPoint(uri, image, bitmap); } diff --git a/javase/src/com/google/zxing/client/j2se/GUIRunner.java b/javase/src/com/google/zxing/client/j2se/GUIRunner.java index 9d26021cd..78e7732d3 100644 --- a/javase/src/com/google/zxing/client/j2se/GUIRunner.java +++ b/javase/src/com/google/zxing/client/j2se/GUIRunner.java @@ -21,7 +21,7 @@ import com.google.zxing.LuminanceSource; import com.google.zxing.MultiFormatReader; import com.google.zxing.ReaderException; import com.google.zxing.Result; -import com.google.zxing.common.GlobalHistogramBinarizer; +import com.google.zxing.common.HybridBinarizer; import java.awt.Container; import java.awt.Dimension; @@ -95,7 +95,7 @@ public final class GUIRunner extends JFrame { return "Could not decode image"; } LuminanceSource source = new BufferedImageLuminanceSource(image); - BinaryBitmap bitmap = new BinaryBitmap(new GlobalHistogramBinarizer(source)); + BinaryBitmap bitmap = new BinaryBitmap(new HybridBinarizer(source)); Result result; try { result = new MultiFormatReader().decode(bitmap); diff --git a/javase/src/com/google/zxing/client/j2se/ImageConverter.java b/javase/src/com/google/zxing/client/j2se/ImageConverter.java index 4b4cd1653..538e59a06 100644 --- a/javase/src/com/google/zxing/client/j2se/ImageConverter.java +++ b/javase/src/com/google/zxing/client/j2se/ImageConverter.java @@ -21,7 +21,7 @@ import com.google.zxing.LuminanceSource; import com.google.zxing.ReaderException; import com.google.zxing.common.BitArray; import com.google.zxing.common.BitMatrix; -import com.google.zxing.common.GlobalHistogramBinarizer; +import com.google.zxing.common.HybridBinarizer; import java.awt.image.BufferedImage; import java.io.File; @@ -96,7 +96,7 @@ public final class ImageConverter { private static void convertImage(URI uri) throws IOException { BufferedImage image = ImageIO.read(uri.toURL()); LuminanceSource source = new BufferedImageLuminanceSource(image); - BinaryBitmap bitmap = new BinaryBitmap(new GlobalHistogramBinarizer(source)); + BinaryBitmap bitmap = new BinaryBitmap(new HybridBinarizer(source)); int width = bitmap.getWidth(); int height = bitmap.getHeight(); diff --git a/zxingorg/src/com/google/zxing/web/DecodeEmailTask.java b/zxingorg/src/com/google/zxing/web/DecodeEmailTask.java index 795e9f971..550ff3f4a 100644 --- a/zxingorg/src/com/google/zxing/web/DecodeEmailTask.java +++ b/zxingorg/src/com/google/zxing/web/DecodeEmailTask.java @@ -23,7 +23,7 @@ import com.google.zxing.Reader; import com.google.zxing.ReaderException; import com.google.zxing.Result; import com.google.zxing.client.j2se.BufferedImageLuminanceSource; -import com.google.zxing.common.GlobalHistogramBinarizer; +import com.google.zxing.common.HybridBinarizer; import java.awt.image.BufferedImage; import java.io.IOException; @@ -147,7 +147,7 @@ final class DecodeEmailTask extends TimerTask { Result result = null; try { LuminanceSource source = new BufferedImageLuminanceSource(image); - BinaryBitmap bitmap = new BinaryBitmap(new GlobalHistogramBinarizer(source)); + BinaryBitmap bitmap = new BinaryBitmap(new HybridBinarizer(source)); result = reader.decode(bitmap, DecodeServlet.HINTS); } catch (ReaderException re) { log.info("Decoding FAILED"); @@ -191,4 +191,4 @@ final class DecodeEmailTask extends TimerTask { new DecodeEmailTask(emailAddress, emailAuthenticator).run(); } -} \ No newline at end of file +} diff --git a/zxingorg/src/com/google/zxing/web/DecodeServlet.java b/zxingorg/src/com/google/zxing/web/DecodeServlet.java index 7803a407f..4ed5f5a39 100644 --- a/zxingorg/src/com/google/zxing/web/DecodeServlet.java +++ b/zxingorg/src/com/google/zxing/web/DecodeServlet.java @@ -27,7 +27,7 @@ import com.google.zxing.Result; import com.google.zxing.client.j2se.BufferedImageLuminanceSource; import com.google.zxing.client.result.ParsedResult; import com.google.zxing.client.result.ResultParser; -import com.google.zxing.common.GlobalHistogramBinarizer; +import com.google.zxing.common.HybridBinarizer; import org.apache.commons.fileupload.FileItem; import org.apache.commons.fileupload.FileUploadException; @@ -78,7 +78,7 @@ import javax.servlet.http.HttpServletResponse; /** * {@link HttpServlet} which decodes images containing barcodes. Given a URL, it will * retrieve the image and decode it. It can also process image files uploaded via POST. - * + * * @author Sean Owen */ public final class DecodeServlet extends HttpServlet { @@ -102,7 +102,7 @@ public final class DecodeServlet extends HttpServlet { possibleFormats.add(BarcodeFormat.ITF); possibleFormats.add(BarcodeFormat.QR_CODE); possibleFormats.add(BarcodeFormat.DATAMATRIX); - possibleFormats.add(BarcodeFormat.PDF417); + possibleFormats.add(BarcodeFormat.PDF417); HINTS.put(DecodeHintType.POSSIBLE_FORMATS, possibleFormats); } @@ -249,7 +249,7 @@ public final class DecodeServlet extends HttpServlet { Result result; try { LuminanceSource source = new BufferedImageLuminanceSource(image); - BinaryBitmap bitmap = new BinaryBitmap(new GlobalHistogramBinarizer(source)); + BinaryBitmap bitmap = new BinaryBitmap(new HybridBinarizer(source)); result = reader.decode(bitmap, HINTS); } catch (ReaderException re) { log.info("DECODE FAILED: " + re.toString());