Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(214)

Unified Diff: xfa/fxbarcode/pdf417/BC_PDF417ECErrorCorrection.cpp

Issue 2048983002: Get rid of NULLs in xfa/fxbarcode/ (Closed) Base URL: https://pdfium.googlesource.com/pdfium@master
Patch Set: nits Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « xfa/fxbarcode/pdf417/BC_PDF417Detector.cpp ('k') | xfa/fxbarcode/pdf417/BC_PDF417ECModulusGF.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: xfa/fxbarcode/pdf417/BC_PDF417ECErrorCorrection.cpp
diff --git a/xfa/fxbarcode/pdf417/BC_PDF417ECErrorCorrection.cpp b/xfa/fxbarcode/pdf417/BC_PDF417ECErrorCorrection.cpp
index 6a1f46fd0224e7ea09d2b0fa5c69107d9b283aa4..aadb1a6ac53536a3b38b1b55c188c26c5ff4911f 100644
--- a/xfa/fxbarcode/pdf417/BC_PDF417ECErrorCorrection.cpp
+++ b/xfa/fxbarcode/pdf417/BC_PDF417ECErrorCorrection.cpp
@@ -26,7 +26,7 @@
#include "xfa/fxbarcode/pdf417/BC_PDF417ECModulusPoly.h"
#include "xfa/fxbarcode/utils.h"
-CBC_PDF417ECModulusGF* CBC_PDF417ECErrorCorrection::m_field = NULL;
+CBC_PDF417ECModulusGF* CBC_PDF417ECErrorCorrection::m_field = nullptr;
void CBC_PDF417ECErrorCorrection::Initialize(int32_t& e) {
m_field =
@@ -137,9 +137,9 @@ CBC_PDF417ECErrorCorrection::runEuclideanAlgorithm(CBC_PDF417ECModulusPoly* a,
CBC_PDF417ECModulusPoly* r = b;
CBC_PDF417ECModulusPoly* tLast = m_field->getZero();
CBC_PDF417ECModulusPoly* t = m_field->getOne();
- CBC_PDF417ECModulusPoly* qtemp = NULL;
- CBC_PDF417ECModulusPoly* rtemp = NULL;
- CBC_PDF417ECModulusPoly* ttemp = NULL;
+ CBC_PDF417ECModulusPoly* qtemp = nullptr;
+ CBC_PDF417ECModulusPoly* rtemp = nullptr;
+ CBC_PDF417ECModulusPoly* ttemp = nullptr;
int32_t i = 0;
int32_t j = 0;
int32_t m = 0;
@@ -261,7 +261,7 @@ CBC_PDF417ECErrorCorrection::runEuclideanAlgorithm(CBC_PDF417ECModulusPoly* a,
}
CBC_PDF417ECModulusPoly* omega = r->multiply(inverse, e);
delete rtemp;
- BC_EXCEPTION_CHECK_ReturnValue(e, NULL);
+ BC_EXCEPTION_CHECK_ReturnValue(e, nullptr);
CFX_ArrayTemplate<CBC_PDF417ECModulusPoly*>* modulusPoly =
new CFX_ArrayTemplate<CBC_PDF417ECModulusPoly*>();
modulusPoly->Add(sigma);
@@ -280,7 +280,7 @@ CFX_Int32Array* CBC_PDF417ECErrorCorrection::findErrorLocations(
result->SetAt(ee, m_field->inverse(i, e));
if (e != BCExceptionNO) {
delete result;
- return NULL;
+ return nullptr;
}
ee++;
}
@@ -288,7 +288,7 @@ CFX_Int32Array* CBC_PDF417ECErrorCorrection::findErrorLocations(
if (ee != numErrors) {
e = BCExceptionChecksumException;
delete result;
- return NULL;
+ return nullptr;
}
return result;
}
@@ -306,7 +306,7 @@ CFX_Int32Array* CBC_PDF417ECErrorCorrection::findErrorMagnitudes(
}
CBC_PDF417ECModulusPoly formalDerivative(m_field,
formalDerivativeCoefficients, e);
- BC_EXCEPTION_CHECK_ReturnValue(e, NULL);
+ BC_EXCEPTION_CHECK_ReturnValue(e, nullptr);
int32_t s = errorLocations.GetSize();
CFX_Int32Array* result = new CFX_Int32Array;
result->SetSize(s);
@@ -314,7 +314,7 @@ CFX_Int32Array* CBC_PDF417ECErrorCorrection::findErrorMagnitudes(
int32_t xiInverse = m_field->inverse(errorLocations[i], e);
if (e != BCExceptionNO) {
delete result;
- return NULL;
+ return nullptr;
}
int32_t numerator =
m_field->subtract(0, errorEvaluator->evaluateAt(xiInverse));
@@ -322,7 +322,7 @@ CFX_Int32Array* CBC_PDF417ECErrorCorrection::findErrorMagnitudes(
m_field->inverse(formalDerivative.evaluateAt(xiInverse), e);
if (e != BCExceptionNO) {
delete result;
- return NULL;
+ return nullptr;
}
result->SetAt(i, m_field->multiply(numerator, denominator));
}
« no previous file with comments | « xfa/fxbarcode/pdf417/BC_PDF417Detector.cpp ('k') | xfa/fxbarcode/pdf417/BC_PDF417ECModulusGF.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698