Fixed things broken in the last commit.

git-svn-id: https://zxing.googlecode.com/svn/trunk@1272 59b500cc-1b3d-0410-9834-0bbf25fbcc57
This commit is contained in:
ralf.kistner@gmail.com 2010-04-02 17:17:31 +00:00
parent d2aacefff2
commit 416b2e58ac
10 changed files with 37 additions and 42 deletions

View file

@ -20,6 +20,7 @@
#include <zxing/common/BitArray.h> #include <zxing/common/BitArray.h>
#include <iostream> #include <iostream>
#include <limits>
using namespace std; using namespace std;
@ -33,7 +34,7 @@ static unsigned int logDigits(unsigned digits) {
} }
return log; return log;
} }
const unsigned int BitArray::bitsPerWord_ = sizeof(unsigned int)*8; const unsigned int BitArray::bitsPerWord_ = numeric_limits<unsigned int>::digits;
const unsigned int BitArray::logBits_ = logDigits(bitsPerWord_); const unsigned int BitArray::logBits_ = logDigits(bitsPerWord_);
const unsigned int BitArray::bitsMask_ = (1 << logBits_) - 1; const unsigned int BitArray::bitsMask_ = (1 << logBits_) - 1;
size_t BitArray::wordsForBits(size_t bits) { size_t BitArray::wordsForBits(size_t bits) {

View file

@ -24,7 +24,6 @@
#include <zxing/common/Counted.h> #include <zxing/common/Counted.h>
#include <zxing/common/IllegalArgumentException.h> #include <zxing/common/IllegalArgumentException.h>
#include <vector> #include <vector>
#include <limits>
#include <iostream> #include <iostream>
namespace zxing { namespace zxing {

View file

@ -39,7 +39,7 @@ unsigned int logDigits(unsigned digits) {
} }
const unsigned int bitsPerWord = sizeof(unsigned int)*8; const unsigned int bitsPerWord = numeric_limits<unsigned int>::digits;
const unsigned int logBits = logDigits(bitsPerWord); const unsigned int logBits = logDigits(bitsPerWord);
const unsigned int bitsMask = (1 << logBits) - 1; const unsigned int bitsMask = (1 << logBits) - 1;

View file

@ -181,7 +181,7 @@ Ref<ResultPointsAndTransitions> Detector::transitionsBetween(Ref<CornerPoint> fr
int fromY = (int) from->getY(); int fromY = (int) from->getY();
int toX = (int) to->getX(); int toX = (int) to->getX();
int toY = (int) to->getY(); int toY = (int) to->getY();
bool steep = labs(toY - fromY) > labs(toX - fromX); bool steep = abs(toY - fromY) > abs(toX - fromX);
if (steep) { if (steep) {
int temp = fromX; int temp = fromX;
fromX = fromY; fromX = fromY;
@ -191,8 +191,8 @@ Ref<ResultPointsAndTransitions> Detector::transitionsBetween(Ref<CornerPoint> fr
toY = temp; toY = temp;
} }
int dx = labs(toX - fromX); int dx = abs(toX - fromX);
int dy = labs(toY - fromY); int dy = abs(toY - fromY);
int error = -dx >> 1; int error = -dx >> 1;
int ystep = fromY < toY ? 1 : -1; int ystep = fromY < toY ? 1 : -1;
int xstep = fromX < toX ? 1 : -1; int xstep = fromX < toX ? 1 : -1;

View file

@ -52,10 +52,10 @@ const char *DecodedBitStreamParser::UTF8 = "UTF-8";
const char *DecodedBitStreamParser::SHIFT_JIS = "SHIFT_JIS"; const char *DecodedBitStreamParser::SHIFT_JIS = "SHIFT_JIS";
const char *DecodedBitStreamParser::EUC_JP = "EUC-JP"; const char *DecodedBitStreamParser::EUC_JP = "EUC-JP";
string DecodedBitStreamParser::convert(const unsigned char *bufIn, size_t nIn, const char *src) { void DecodedBitStreamParser::append(std::string &result, const unsigned char *bufIn, size_t nIn, const char *src) {
#ifndef NO_ICONV #ifndef NO_ICONV
if (nIn == 0) { if (nIn == 0) {
return string(); return;
} }
iconv_t cd = iconv_open(UTF8, src); iconv_t cd = iconv_open(UTF8, src);
@ -79,15 +79,14 @@ string DecodedBitStreamParser::convert(const unsigned char *bufIn, size_t nIn, c
int nResult = maxOut - nTo; int nResult = maxOut - nTo;
bufOut[nResult] = '\0'; bufOut[nResult] = '\0';
string result((const char *)bufOut); result.append((const char *)bufOut);
delete[] bufOut; delete[] bufOut;
return result;
#else #else
return string((const char *)bufIn, nIn); result.append((const char *)bufIn, nIn);
#endif #endif
} }
string DecodedBitStreamParser::decodeKanjiSegment(Ref<BitSource> bits, int count) { void DecodedBitStreamParser::decodeKanjiSegment(Ref<BitSource> bits, std::string &result, int count) {
// Each character will require 2 bytes. Read the characters as 2-byte pairs // Each character will require 2 bytes. Read the characters as 2-byte pairs
// and decode as Shift_JIS afterwards // and decode as Shift_JIS afterwards
size_t nBytes = 2 * count; size_t nBytes = 2 * count;
@ -111,12 +110,11 @@ string DecodedBitStreamParser::decodeKanjiSegment(Ref<BitSource> bits, int count
count--; count--;
} }
string result = convert(buffer, nBytes, SHIFT_JIS); append(result, buffer, nBytes, SHIFT_JIS);
delete[] buffer; delete[] buffer;
return result;
} }
string DecodedBitStreamParser::decodeByteSegment(Ref<BitSource> bits, int count) { void DecodedBitStreamParser::decodeByteSegment(Ref<BitSource> bits, std::string &result, int count) {
int nBytes = count; int nBytes = count;
unsigned char* readBytes = new unsigned char[nBytes]; unsigned char* readBytes = new unsigned char[nBytes];
if (count << 3 > bits->available()) { if (count << 3 > bits->available()) {
@ -134,12 +132,11 @@ string DecodedBitStreamParser::decodeByteSegment(Ref<BitSource> bits, int count)
// Shift_JIS -- without anything like an ECI designator to // Shift_JIS -- without anything like an ECI designator to
// give a hint. // give a hint.
const char *encoding = guessEncoding(readBytes, nBytes); const char *encoding = guessEncoding(readBytes, nBytes);
string result = convert(readBytes, nBytes, encoding); append(result, readBytes, nBytes, encoding);
delete[] readBytes; delete[] readBytes;
return result;
} }
string DecodedBitStreamParser::decodeNumericSegment(Ref<BitSource> bits, int count) { void DecodedBitStreamParser::decodeNumericSegment(Ref<BitSource> bits, std::string &result, int count) {
int nBytes = count; int nBytes = count;
unsigned char* bytes = new unsigned char[nBytes]; unsigned char* bytes = new unsigned char[nBytes];
int i = 0; int i = 0;
@ -180,12 +177,11 @@ string DecodedBitStreamParser::decodeNumericSegment(Ref<BitSource> bits, int cou
} }
bytes[i++] = ALPHANUMERIC_CHARS[digitBits]; bytes[i++] = ALPHANUMERIC_CHARS[digitBits];
} }
string result = convert(bytes, nBytes, ASCII); append(result, bytes, nBytes, ASCII);
delete[] bytes; delete[] bytes;
return result;
} }
string DecodedBitStreamParser::decodeAlphanumericSegment(Ref<BitSource> bits, int count) { void DecodedBitStreamParser::decodeAlphanumericSegment(Ref<BitSource> bits, std::string &result, int count) {
int nBytes = count; int nBytes = count;
unsigned char* bytes = new unsigned char[nBytes]; unsigned char* bytes = new unsigned char[nBytes];
int i = 0; int i = 0;
@ -199,9 +195,8 @@ string DecodedBitStreamParser::decodeAlphanumericSegment(Ref<BitSource> bits, in
if (count == 1) { if (count == 1) {
bytes[i++] = ALPHANUMERIC_CHARS[bits->readBits(6)]; bytes[i++] = ALPHANUMERIC_CHARS[bits->readBits(6)];
} }
string result = convert(bytes, nBytes, ASCII); append(result, bytes, nBytes, ASCII);
delete[] bytes; delete[] bytes;
return result;
} }
const char * const char *
@ -273,13 +268,13 @@ string DecodedBitStreamParser::decode(ArrayRef<unsigned char> bytes, Version *ve
// How many characters will follow, encoded in this mode? // How many characters will follow, encoded in this mode?
int count = bits->readBits(mode->getCharacterCountBits(version)); int count = bits->readBits(mode->getCharacterCountBits(version));
if (mode == &Mode::NUMERIC) { if (mode == &Mode::NUMERIC) {
result = decodeNumericSegment(bits, count); decodeNumericSegment(bits, result, count);
} else if (mode == &Mode::ALPHANUMERIC) { } else if (mode == &Mode::ALPHANUMERIC) {
result = decodeAlphanumericSegment(bits, count); decodeAlphanumericSegment(bits, result, count);
} else if (mode == &Mode::BYTE) { } else if (mode == &Mode::BYTE) {
result = decodeByteSegment(bits, count); decodeByteSegment(bits, result, count);
} else if (mode == &Mode::KANJI) { } else if (mode == &Mode::KANJI) {
result = decodeKanjiSegment(bits, count); decodeKanjiSegment(bits, result, count);
} else { } else {
throw ReaderException("Unsupported mode indicator"); throw ReaderException("Unsupported mode indicator");
} }

View file

@ -43,12 +43,12 @@ private:
static const char *SHIFT_JIS; static const char *SHIFT_JIS;
static const char *EUC_JP; static const char *EUC_JP;
static std::string decodeKanjiSegment(Ref<BitSource> bits, int count); static void decodeKanjiSegment(Ref<BitSource> bits, std::string &result, int count);
static std::string decodeByteSegment(Ref<BitSource> bits, int count); static void decodeByteSegment(Ref<BitSource> bits, std::string &result, int count);
static std::string decodeAlphanumericSegment(Ref<BitSource> bits, int count); static void decodeAlphanumericSegment(Ref<BitSource> bits, std::string &result, int count);
static std::string decodeNumericSegment(Ref<BitSource> bits, int count); static void decodeNumericSegment(Ref<BitSource> bits, std::string &result, int count);
static const char *guessEncoding(unsigned char *bytes, int length); static const char *guessEncoding(unsigned char *bytes, int length);
static std::string convert(const unsigned char *bufIn, size_t nIn, const char *src); static void append(std::string &ost, const unsigned char *bufIn, size_t nIn, const char *src);
public: public:
static std::string decode(ArrayRef<unsigned char> bytes, Version *version); static std::string decode(ArrayRef<unsigned char> bytes, Version *version);

View file

@ -37,7 +37,7 @@ float AlignmentPatternFinder::centerFromEnd(vector<int> &stateCount, int end) {
bool AlignmentPatternFinder::foundPatternCross(vector<int> &stateCount) { bool AlignmentPatternFinder::foundPatternCross(vector<int> &stateCount) {
float maxVariance = moduleSize_ / 2.0f; float maxVariance = moduleSize_ / 2.0f;
for (size_t i = 0; i < 3; i++) { for (size_t i = 0; i < 3; i++) {
if (labs(moduleSize_ - stateCount[i]) >= maxVariance) { if (abs(moduleSize_ - stateCount[i]) >= maxVariance) {
return false; return false;
} }
} }
@ -86,7 +86,7 @@ float AlignmentPatternFinder::crossCheckVertical(size_t startI, size_t centerJ,
} }
int stateCountTotal = stateCount[0] + stateCount[1] + stateCount[2]; int stateCountTotal = stateCount[0] + stateCount[1] + stateCount[2];
if (5 * labs(stateCountTotal - originalStateCountTotal) >= 2 * originalStateCountTotal) { if (5 * abs(stateCountTotal - originalStateCountTotal) >= 2 * originalStateCountTotal) {
return NAN; return NAN;
} }

View file

@ -201,7 +201,7 @@ float Detector::sizeOfBlackWhiteBlackRunBothWays(int fromX, int fromY, int toX,
float Detector::sizeOfBlackWhiteBlackRun(int fromX, int fromY, int toX, int toY) { float Detector::sizeOfBlackWhiteBlackRun(int fromX, int fromY, int toX, int toY) {
// Mild variant of Bresenham's algorithm; // Mild variant of Bresenham's algorithm;
// see http://en.wikipedia.org/wiki/Bresenham's_line_algorithm // see http://en.wikipedia.org/wiki/Bresenham's_line_algorithm
bool steep = labs(toY - fromY) > labs(toX - fromX); bool steep = abs(toY - fromY) > abs(toX - fromX);
if (steep) { if (steep) {
int temp = fromX; int temp = fromX;
fromX = fromY; fromX = fromY;
@ -211,8 +211,8 @@ float Detector::sizeOfBlackWhiteBlackRun(int fromX, int fromY, int toX, int toY)
toY = temp; toY = temp;
} }
int dx = labs(toX - fromX); int dx = abs(toX - fromX);
int dy = labs(toY - fromY); int dy = abs(toY - fromY);
int error = -dx >> 1; int error = -dx >> 1;
int ystep = fromY < toY ? 1 : -1; int ystep = fromY < toY ? 1 : -1;
int xstep = fromX < toX ? 1 : -1; int xstep = fromX < toX ? 1 : -1;

View file

@ -140,7 +140,7 @@ float FinderPatternFinder::crossCheckVertical(size_t startI, size_t centerJ, int
// If we found a finder-pattern-like section, but its size is more than 40% different than // If we found a finder-pattern-like section, but its size is more than 40% different than
// the original, assume it's a false positive // the original, assume it's a false positive
int stateCountTotal = stateCount[0] + stateCount[1] + stateCount[2] + stateCount[3] + stateCount[4]; int stateCountTotal = stateCount[0] + stateCount[1] + stateCount[2] + stateCount[3] + stateCount[4];
if (5 * labs(stateCountTotal - originalStateCountTotal) >= 2 * originalStateCountTotal) { if (5 * abs(stateCountTotal - originalStateCountTotal) >= 2 * originalStateCountTotal) {
return NAN; return NAN;
} }
@ -204,7 +204,7 @@ float FinderPatternFinder::crossCheckHorizontal(size_t startJ, size_t centerI, i
// If we found a finder-pattern-like section, but its size is significantly different than // If we found a finder-pattern-like section, but its size is significantly different than
// the original, assume it's a false positive // the original, assume it's a false positive
int stateCountTotal = stateCount[0] + stateCount[1] + stateCount[2] + stateCount[3] + stateCount[4]; int stateCountTotal = stateCount[0] + stateCount[1] + stateCount[2] + stateCount[3] + stateCount[4];
if (5 * labs(stateCountTotal - originalStateCountTotal) >= originalStateCountTotal) { if (5 * abs(stateCountTotal - originalStateCountTotal) >= originalStateCountTotal) {
return NAN; return NAN;
} }

View file

@ -114,7 +114,7 @@ Point QREdgeDetector::endOfReverseBlackWhiteBlackRun(const BitMatrix& image, Poi
int toX = (int)to.x; int toX = (int)to.x;
int toY = (int)to.y; int toY = (int)to.y;
bool steep = labs(toY - fromY) > labs(toX - fromX); bool steep = abs(toY - fromY) > abs(toX - fromX);
if (steep) { if (steep) {
int temp = fromX; int temp = fromX;
fromX = fromY; fromX = fromY;
@ -124,8 +124,8 @@ Point QREdgeDetector::endOfReverseBlackWhiteBlackRun(const BitMatrix& image, Poi
toY = temp; toY = temp;
} }
int dx = labs(toX - fromX); int dx = abs(toX - fromX);
int dy = labs(toY - fromY); int dy = abs(toY - fromY);
int error = -dx >> 1; int error = -dx >> 1;
int ystep = fromY < toY ? -1 : 1; int ystep = fromY < toY ? -1 : 1;
int xstep = fromX < toX ? -1 : 1; int xstep = fromX < toX ? -1 : 1;