summaryrefslogtreecommitdiff
path: root/xfa
diff options
context:
space:
mode:
authorJun Fang <jun_fang@foxitsoftware.com>2015-11-03 19:01:30 -0800
committerJun Fang <jun_fang@foxitsoftware.com>2015-11-03 19:01:30 -0800
commit4a8eeb74d38a1589c21ec06b68262d7795dd8f71 (patch)
tree9a78f3fb9223adf2a3d26eb972da4461c3d22b5f /xfa
parent844ccfeab312ed294059de94118a40336df875d7 (diff)
downloadpdfium-4a8eeb74d38a1589c21ec06b68262d7795dd8f71.tar.xz
Remove unused variables
BUG=pdfium:261 R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1413343003 .
Diffstat (limited to 'xfa')
-rw-r--r--xfa/src/fwl/src/core/fwl_noteimp.cpp2
-rw-r--r--xfa/src/fxbarcode/BC_BarCode.cpp1
-rw-r--r--xfa/src/fxbarcode/common/reedsolomon/BC_ReedSolomonGF256Poly.cpp1
-rw-r--r--xfa/src/fxbarcode/datamatrix/BC_ASCIIEncoder.cpp1
-rw-r--r--xfa/src/fxbarcode/datamatrix/BC_ErrorCorrection.cpp7
-rw-r--r--xfa/src/fxbarcode/oned/BC_OneDReader.cpp1
-rw-r--r--xfa/src/fxbarcode/oned/BC_OneDimReader.cpp2
-rw-r--r--xfa/src/fxbarcode/oned/BC_OneDimWriter.cpp4
-rw-r--r--xfa/src/fxbarcode/oned/BC_OnedCode128Writer.cpp1
-rw-r--r--xfa/src/fxbarcode/oned/BC_OnedCode39Reader.cpp1
-rw-r--r--xfa/src/fxbarcode/oned/BC_OnedEAN8Writer.cpp4
-rw-r--r--xfa/src/fxbarcode/pdf417/BC_PDF417.cpp1
-rw-r--r--xfa/src/fxbarcode/pdf417/BC_PDF417ECErrorCorrection.cpp1
-rw-r--r--xfa/src/fxbarcode/pdf417/BC_PDF417HighLevelEncoder.cpp2
-rw-r--r--xfa/src/fxbarcode/pdf417/BC_PDF417ScanningDecoder.cpp4
-rw-r--r--xfa/src/fxbarcode/qrcode/BC_QRAlignmentPatternFinder.cpp1
-rw-r--r--xfa/src/fxbarcode/qrcode/BC_QRCoderDecoder.cpp1
-rw-r--r--xfa/src/fxbarcode/qrcode/BC_QRCoderEncoder.cpp2
-rw-r--r--xfa/src/fxbarcode/qrcode/BC_QRDecodedBitStreamParser.cpp1
-rw-r--r--xfa/src/fxfa/src/app/xfa_ffdocview.cpp2
-rw-r--r--xfa/src/fxfa/src/app/xfa_fffield.cpp1
-rw-r--r--xfa/src/fxfa/src/app/xfa_fftextedit.cpp1
-rw-r--r--xfa/src/fxfa/src/app/xfa_ffwidget.cpp3
-rw-r--r--xfa/src/fxfa/src/app/xfa_ffwidgetacc.cpp1
-rw-r--r--xfa/src/fxfa/src/app/xfa_ffwidgethandler.cpp13
-rw-r--r--xfa/src/fxfa/src/app/xfa_textlayout.cpp2
-rw-r--r--xfa/src/fxfa/src/common/xfa_utils.h1
-rw-r--r--xfa/src/fxfa/src/fm2js/xfa_fm2jscontext.cpp20
-rw-r--r--xfa/src/fxfa/src/parser/xfa_document_datamerger_imp.cpp5
-rw-r--r--xfa/src/fxfa/src/parser/xfa_layout_itemlayout.cpp3
-rw-r--r--xfa/src/fxfa/src/parser/xfa_layout_pagemgr_new.cpp6
-rw-r--r--xfa/src/fxfa/src/parser/xfa_object_imp.cpp11
-rw-r--r--xfa/src/fxfa/src/parser/xfa_objectacc_imp.cpp2
-rw-r--r--xfa/src/fxfa/src/parser/xfa_parser_imp.cpp5
-rw-r--r--xfa/src/fxfa/src/parser/xfa_script_hostpseudomodel.cpp1
-rw-r--r--xfa/src/fxfa/src/parser/xfa_script_layoutpseudomodel.cpp1
-rw-r--r--xfa/src/fxfa/src/parser/xfa_script_resolveprocessor.cpp4
-rw-r--r--xfa/src/fxfa/src/parser/xfa_utils_imp.cpp1
38 files changed, 22 insertions, 99 deletions
diff --git a/xfa/src/fwl/src/core/fwl_noteimp.cpp b/xfa/src/fwl/src/core/fwl_noteimp.cpp
index 6bf017d736..ab7546c0f0 100644
--- a/xfa/src/fwl/src/core/fwl_noteimp.cpp
+++ b/xfa/src/fwl/src/core/fwl_noteimp.cpp
@@ -218,8 +218,6 @@ FX_BOOL CFWL_NoteDriver::SetFocus(IFWL_Widget* pFocus, FX_BOOL bNotify) {
IFWL_Widget* pPrev = m_pFocus;
m_pFocus = pFocus;
if (pPrev) {
- IFWL_Widget* pWidget =
- FWL_GetWidgetMgr()->GetWidget(m_pFocus, FWL_WGTRELATION_SystemForm);
CFWL_MsgKillFocus ms;
ms.m_pDstTarget = pPrev;
ms.m_pSrcTarget = pPrev;
diff --git a/xfa/src/fxbarcode/BC_BarCode.cpp b/xfa/src/fxbarcode/BC_BarCode.cpp
index bb7ed6b1dc..2bc0559b12 100644
--- a/xfa/src/fxbarcode/BC_BarCode.cpp
+++ b/xfa/src/fxbarcode/BC_BarCode.cpp
@@ -785,7 +785,6 @@ void CBC_PDF417I::SetTruncated(FX_BOOL truncated) {
FX_BOOL CBC_PDF417I::Encode(const CFX_WideStringC& contents,
FX_BOOL isDevice,
int32_t& e) {
- BCFORMAT format = BCFORMAT_PDF_417;
int32_t outWidth = 0;
int32_t outHeight = 0;
uint8_t* data = ((CBC_PDF417Writer*)m_pBCWriter)
diff --git a/xfa/src/fxbarcode/common/reedsolomon/BC_ReedSolomonGF256Poly.cpp b/xfa/src/fxbarcode/common/reedsolomon/BC_ReedSolomonGF256Poly.cpp
index 127a20b4f9..b94508f900 100644
--- a/xfa/src/fxbarcode/common/reedsolomon/BC_ReedSolomonGF256Poly.cpp
+++ b/xfa/src/fxbarcode/common/reedsolomon/BC_ReedSolomonGF256Poly.cpp
@@ -226,7 +226,6 @@ CFX_PtrArray* CBC_ReedSolomonGF256Poly::Divide(CBC_ReedSolomonGF256Poly* other,
int32_t inverseDenominatorLeadingTeam =
m_field->Inverse(denominatorLeadingTerm, e);
BC_EXCEPTION_CHECK_ReturnValue(e, NULL);
- FX_BOOL bFirst = TRUE;
while (remainder->GetDegree() >= other->GetDegree() && !remainder->IsZero()) {
int32_t degreeDifference = remainder->GetDegree() - other->GetDegree();
int32_t scale =
diff --git a/xfa/src/fxbarcode/datamatrix/BC_ASCIIEncoder.cpp b/xfa/src/fxbarcode/datamatrix/BC_ASCIIEncoder.cpp
index 3931206140..3663b9f84e 100644
--- a/xfa/src/fxbarcode/datamatrix/BC_ASCIIEncoder.cpp
+++ b/xfa/src/fxbarcode/datamatrix/BC_ASCIIEncoder.cpp
@@ -91,7 +91,6 @@ FX_WCHAR CBC_ASCIIEncoder::encodeASCIIDigits(FX_WCHAR digit1,
if (CBC_HighLevelEncoder::isDigit(digit1) &&
CBC_HighLevelEncoder::isDigit(digit2)) {
int32_t num = (digit1 - 48) * 10 + (digit2 - 48);
- FX_WCHAR a = (FX_WCHAR)(num + 130);
return (FX_WCHAR)(num + 130);
}
e = BCExceptionIllegalArgumentNotGigits;
diff --git a/xfa/src/fxbarcode/datamatrix/BC_ErrorCorrection.cpp b/xfa/src/fxbarcode/datamatrix/BC_ErrorCorrection.cpp
index 32dd416522..ffb52350c9 100644
--- a/xfa/src/fxbarcode/datamatrix/BC_ErrorCorrection.cpp
+++ b/xfa/src/fxbarcode/datamatrix/BC_ErrorCorrection.cpp
@@ -151,16 +151,9 @@ CFX_WideString CBC_ErrorCorrection::createECCBlock(CFX_WideString codewords,
FX_WORD* ecc = FX_Alloc(FX_WORD, numECWords);
FXSYS_memset(ecc, 0, numECWords * sizeof(FX_WORD));
for (int32_t l = start; l < start + len; l++) {
- uint8_t A = (uint8_t)codewords.GetAt(l);
- FX_WORD B = ecc[numECWords - 1];
FX_WORD m = ecc[numECWords - 1] ^ codewords.GetAt(l);
for (int32_t k = numECWords - 1; k > 0; k--) {
if (m != 0 && FACTORS[table][k] != 0) {
- int32_t a = LOG[FACTORS[table][k]];
- int32_t b = ALOG[(LOG[m] + LOG[FACTORS[table][k]]) % 255];
- FX_WORD c = ecc[k - 1];
- FX_WORD D =
- (ecc[k - 1] ^ ALOG[(LOG[m] + LOG[FACTORS[table][k]]) % 255]);
ecc[k] = (FX_WORD)(ecc[k - 1] ^
ALOG[(LOG[m] + LOG[FACTORS[table][k]]) % 255]);
} else {
diff --git a/xfa/src/fxbarcode/oned/BC_OneDReader.cpp b/xfa/src/fxbarcode/oned/BC_OneDReader.cpp
index 2338d0d086..6e4798b666 100644
--- a/xfa/src/fxbarcode/oned/BC_OneDReader.cpp
+++ b/xfa/src/fxbarcode/oned/BC_OneDReader.cpp
@@ -44,7 +44,6 @@ CFX_ByteString CBC_OneDReader::Decode(CBC_BinaryBitmap* image,
CFX_ByteString CBC_OneDReader::DeDecode(CBC_BinaryBitmap* image,
int32_t hints,
int32_t& e) {
- int32_t width = image->GetWidth();
int32_t height = image->GetHeight();
CBC_CommonBitArray* row = NULL;
int32_t middle = height >> 1;
diff --git a/xfa/src/fxbarcode/oned/BC_OneDimReader.cpp b/xfa/src/fxbarcode/oned/BC_OneDimReader.cpp
index 77c34802b5..67bfd80995 100644
--- a/xfa/src/fxbarcode/oned/BC_OneDimReader.cpp
+++ b/xfa/src/fxbarcode/oned/BC_OneDimReader.cpp
@@ -87,7 +87,7 @@ CFX_ByteString CBC_OneDimReader::DecodeRow(int32_t rowNumber,
int32_t hints,
int32_t& e) {
CFX_ByteString result;
- int32_t endStart = DecodeMiddle(row, startGuardRange, result, e);
+ DecodeMiddle(row, startGuardRange, result, e);
BC_EXCEPTION_CHECK_ReturnValue(e, "");
FX_BOOL b = CheckChecksum(result, e);
BC_EXCEPTION_CHECK_ReturnValue(e, "");
diff --git a/xfa/src/fxbarcode/oned/BC_OneDimWriter.cpp b/xfa/src/fxbarcode/oned/BC_OneDimWriter.cpp
index 8332986bae..11d1021aa2 100644
--- a/xfa/src/fxbarcode/oned/BC_OneDimWriter.cpp
+++ b/xfa/src/fxbarcode/oned/BC_OneDimWriter.cpp
@@ -197,7 +197,7 @@ void CBC_OneDimWriter::ShowDeviceChars(CFX_RenderDevice* device,
if (matrix != NULL) {
affine_matrix.Concat(*matrix);
}
- FX_BOOL ret = device->DrawNormalText(
+ device->DrawNormalText(
str.GetLength(), pCharPos, m_pFont, CFX_GEModule::Get()->GetFontCache(),
(FX_FLOAT)iFontSize, (CFX_AffineMatrix*)&affine_matrix, m_fontColor,
FXTEXT_CLEARTYPE);
@@ -216,7 +216,7 @@ void CBC_OneDimWriter::ShowBitmapChars(CFX_DIBitmap* pOutBitmap,
FX_RECT geRect(0, 0, (int)geWidth, iTextHeight);
ge.FillRect(&geRect, m_backgroundColor);
CFX_AffineMatrix affine_matrix(1.0, 0.0, 0.0, -1.0, 0.0, (FX_FLOAT)iFontSize);
- FX_BOOL ret = ge.DrawNormalText(
+ ge.DrawNormalText(
str.GetLength(), pCharPos, m_pFont, CFX_GEModule::Get()->GetFontCache(),
(FX_FLOAT)iFontSize, (CFX_AffineMatrix*)&affine_matrix, m_fontColor,
FXTEXT_CLEARTYPE);
diff --git a/xfa/src/fxbarcode/oned/BC_OnedCode128Writer.cpp b/xfa/src/fxbarcode/oned/BC_OnedCode128Writer.cpp
index 2c30024dd1..bdea697841 100644
--- a/xfa/src/fxbarcode/oned/BC_OnedCode128Writer.cpp
+++ b/xfa/src/fxbarcode/oned/BC_OnedCode128Writer.cpp
@@ -145,7 +145,6 @@ FX_BOOL CBC_OnedCode128Writer::IsDigits(const CFX_ByteString& contents,
uint8_t* CBC_OnedCode128Writer::Encode(const CFX_ByteString& contents,
int32_t& outLength,
int32_t& e) {
- int32_t length = contents.GetLength();
if (contents.GetLength() < 1 || contents.GetLength() > 80) {
e = BCExceptionContentsLengthShouldBetween1and80;
return NULL;
diff --git a/xfa/src/fxbarcode/oned/BC_OnedCode39Reader.cpp b/xfa/src/fxbarcode/oned/BC_OnedCode39Reader.cpp
index 9caa0cb43a..43bd696379 100644
--- a/xfa/src/fxbarcode/oned/BC_OnedCode39Reader.cpp
+++ b/xfa/src/fxbarcode/oned/BC_OnedCode39Reader.cpp
@@ -92,7 +92,6 @@ CFX_ByteString CBC_OnedCode39Reader::DecodeRow(int32_t rowNumber,
for (int32_t j = 0; j < counters.GetSize(); j++) {
lastPatternSize += counters[j];
}
- int32_t whiteSpaceAfterEnd = nextStart - lastStart - lastPatternSize;
if (m_usingCheckDigit) {
int32_t max = result.GetLength() - 1;
int32_t total = 0;
diff --git a/xfa/src/fxbarcode/oned/BC_OnedEAN8Writer.cpp b/xfa/src/fxbarcode/oned/BC_OnedEAN8Writer.cpp
index 0c551ccc6b..74ef0fb1f4 100644
--- a/xfa/src/fxbarcode/oned/BC_OnedEAN8Writer.cpp
+++ b/xfa/src/fxbarcode/oned/BC_OnedEAN8Writer.cpp
@@ -229,7 +229,7 @@ void CBC_OnedEAN8Writer::ShowChars(const CFX_WideStringC& contents,
1.0, 0.0, 0.0, -1.0, (FX_FLOAT)leftPosition * m_outputHScale,
(FX_FLOAT)(m_Height - iTextHeight + iFontSize));
affine_matrix1.Concat(*matrix);
- FX_BOOL ret = device->DrawNormalText(
+ device->DrawNormalText(
iLen, pCharPos, m_pFont, CFX_GEModule::Get()->GetFontCache(),
(FX_FLOAT)iFontSize, (CFX_AffineMatrix*)&affine_matrix1, m_fontColor,
FXTEXT_CLEARTYPE);
@@ -257,7 +257,7 @@ void CBC_OnedEAN8Writer::ShowChars(const CFX_WideStringC& contents,
if (matrix != NULL) {
affine_matrix1.Concat(*matrix);
}
- FX_BOOL ret = device->DrawNormalText(
+ device->DrawNormalText(
iLen, pCharPos + 4, m_pFont, CFX_GEModule::Get()->GetFontCache(),
(FX_FLOAT)iFontSize, (CFX_AffineMatrix*)&affine_matrix1, m_fontColor,
FXTEXT_CLEARTYPE);
diff --git a/xfa/src/fxbarcode/pdf417/BC_PDF417.cpp b/xfa/src/fxbarcode/pdf417/BC_PDF417.cpp
index f38758f4bc..a3db95ddf5 100644
--- a/xfa/src/fxbarcode/pdf417/BC_PDF417.cpp
+++ b/xfa/src/fxbarcode/pdf417/BC_PDF417.cpp
@@ -506,7 +506,6 @@ void CBC_PDF417::encodeLowLevel(CFX_WideString fullCodewords,
for (int32_t y = 0; y < r; y++) {
int32_t cluster = y % 3;
logic->startRow();
- int32_t a = START_PATTERN;
encodeChar(START_PATTERN, 17, logic->getCurrentRow());
int32_t left;
int32_t right;
diff --git a/xfa/src/fxbarcode/pdf417/BC_PDF417ECErrorCorrection.cpp b/xfa/src/fxbarcode/pdf417/BC_PDF417ECErrorCorrection.cpp
index a0d8b697d0..9e0351fd35 100644
--- a/xfa/src/fxbarcode/pdf417/BC_PDF417ECErrorCorrection.cpp
+++ b/xfa/src/fxbarcode/pdf417/BC_PDF417ECErrorCorrection.cpp
@@ -284,7 +284,6 @@ CFX_PtrArray* CBC_PDF417ECErrorCorrection::runEuclideanAlgorithm(
ttemp = t;
j++;
}
- int32_t aa = t->getCoefficient(1);
int32_t sigmaTildeAtZero = t->getCoefficient(0);
if (sigmaTildeAtZero == 0) {
e = BCExceptionChecksumException;
diff --git a/xfa/src/fxbarcode/pdf417/BC_PDF417HighLevelEncoder.cpp b/xfa/src/fxbarcode/pdf417/BC_PDF417HighLevelEncoder.cpp
index 80694b91f3..db321bc846 100644
--- a/xfa/src/fxbarcode/pdf417/BC_PDF417HighLevelEncoder.cpp
+++ b/xfa/src/fxbarcode/pdf417/BC_PDF417HighLevelEncoder.cpp
@@ -203,8 +203,6 @@ int32_t CBC_PDF417HighLevelEncoder::encodeText(CFX_WideString msg,
break;
case SUBMODE_MIXED:
if (isMixed(ch)) {
- FX_WCHAR a = MIXED[ch];
- int32_t b = (int32_t)a;
tmp += MIXED[ch];
} else {
if (isAlphaUpper(ch)) {
diff --git a/xfa/src/fxbarcode/pdf417/BC_PDF417ScanningDecoder.cpp b/xfa/src/fxbarcode/pdf417/BC_PDF417ScanningDecoder.cpp
index 1c12c6402f..a34213c153 100644
--- a/xfa/src/fxbarcode/pdf417/BC_PDF417ScanningDecoder.cpp
+++ b/xfa/src/fxbarcode/pdf417/BC_PDF417ScanningDecoder.cpp
@@ -641,7 +641,6 @@ int32_t CBC_PDF417ScanningDecoder::adjustCodewordStartColumn(
int32_t correctedStartColumn = codewordStartColumn;
int32_t increment = leftToRight ? -1 : 1;
for (int32_t i = 0; i < 2; i++) {
- FX_BOOL l = image->Get(correctedStartColumn, imageRow);
while (((leftToRight && correctedStartColumn >= minColumn) ||
(!leftToRight && correctedStartColumn < maxColumn)) &&
leftToRight == image->Get(correctedStartColumn, imageRow)) {
@@ -672,8 +671,7 @@ CBC_CommonDecoderResult* CBC_PDF417ScanningDecoder::decodeCodewords(
return NULL;
}
int32_t numECCodewords = 1 << (ecLevel + 1);
- int32_t correctedErrorsCount =
- correctErrors(codewords, erasures, numECCodewords, e);
+ correctErrors(codewords, erasures, numECCodewords, e);
BC_EXCEPTION_CHECK_ReturnValue(e, NULL);
verifyCodewordCount(codewords, numECCodewords, e);
BC_EXCEPTION_CHECK_ReturnValue(e, NULL);
diff --git a/xfa/src/fxbarcode/qrcode/BC_QRAlignmentPatternFinder.cpp b/xfa/src/fxbarcode/qrcode/BC_QRAlignmentPatternFinder.cpp
index 51bfa32dd0..f504517d44 100644
--- a/xfa/src/fxbarcode/qrcode/BC_QRAlignmentPatternFinder.cpp
+++ b/xfa/src/fxbarcode/qrcode/BC_QRAlignmentPatternFinder.cpp
@@ -131,7 +131,6 @@ FX_FLOAT CBC_QRAlignmentPatternFinder::CrossCheckVertical(
int32_t centerJ,
int32_t maxCount,
int32_t originalStateCountTotal) {
- CBC_CommonBitMatrix* image = m_image;
int32_t maxI = m_image->GetHeight();
CFX_Int32Array stateCount;
stateCount.Copy(m_crossCheckStateCount);
diff --git a/xfa/src/fxbarcode/qrcode/BC_QRCoderDecoder.cpp b/xfa/src/fxbarcode/qrcode/BC_QRCoderDecoder.cpp
index e78d266277..78e7a89f18 100644
--- a/xfa/src/fxbarcode/qrcode/BC_QRCoderDecoder.cpp
+++ b/xfa/src/fxbarcode/qrcode/BC_QRCoderDecoder.cpp
@@ -83,7 +83,6 @@ CBC_CommonDecoderResult* CBC_QRCoderDecoder::Decode(CBC_CommonBitMatrix* bits,
totalBytes += ((CBC_QRDataBlock*)((*dataBlocks)[i]))->GetNumDataCodewords();
}
CFX_ByteArray resultBytes;
- int32_t resultOffset = 0;
for (int32_t j = 0; j < dataBlocks->GetSize(); j++) {
CBC_QRDataBlock* dataBlock = (CBC_QRDataBlock*)((*dataBlocks)[j]);
CFX_ByteArray* codewordBytes = dataBlock->GetCodewords();
diff --git a/xfa/src/fxbarcode/qrcode/BC_QRCoderEncoder.cpp b/xfa/src/fxbarcode/qrcode/BC_QRCoderEncoder.cpp
index 6488619ce1..bbdaa0b93b 100644
--- a/xfa/src/fxbarcode/qrcode/BC_QRCoderEncoder.cpp
+++ b/xfa/src/fxbarcode/qrcode/BC_QRCoderEncoder.cpp
@@ -776,7 +776,7 @@ void CBC_QRCoderEncoder::AppendKanjiBytes(const CFX_ByteString& content,
CBC_QRCoderBitVector* bits,
int32_t& e) {
CFX_ByteArray bytes;
- FX_DWORD value = 0, h = 0;
+ FX_DWORD value = 0;
for (int32_t i = 0; i < bytes.GetSize(); i += 2) {
value = (FX_DWORD)((uint8_t)(content[i] << 8) | (uint8_t)content[i + 1]);
if (value <= 0x9ffc && value >= 0x8140) {
diff --git a/xfa/src/fxbarcode/qrcode/BC_QRDecodedBitStreamParser.cpp b/xfa/src/fxbarcode/qrcode/BC_QRDecodedBitStreamParser.cpp
index 643b15bdfd..26180ceff4 100644
--- a/xfa/src/fxbarcode/qrcode/BC_QRDecodedBitStreamParser.cpp
+++ b/xfa/src/fxbarcode/qrcode/BC_QRDecodedBitStreamParser.cpp
@@ -114,7 +114,6 @@ void CBC_QRDecodedBitStreamParser::DecodeGBKSegment(CBC_CommonBitSource* bits,
int32_t count,
int32_t& e) {
CFX_ByteString buffer;
- int32_t offset = 0;
while (count > 0) {
int32_t twoBytes = bits->ReadBits(13, e);
BC_EXCEPTION_CHECK_ReturnVoid(e);
diff --git a/xfa/src/fxfa/src/app/xfa_ffdocview.cpp b/xfa/src/fxfa/src/app/xfa_ffdocview.cpp
index 1b3432a6cc..8f9265a54d 100644
--- a/xfa/src/fxfa/src/app/xfa_ffdocview.cpp
+++ b/xfa/src/fxfa/src/app/xfa_ffdocview.cpp
@@ -792,8 +792,6 @@ void CXFA_FFDocView::RunBindItems() {
CFX_WideString wsValue, wsLabel;
FX_DWORD uValueHash = FX_HashCode_String_GetW(CFX_WideString(wsValueRef),
wsValueRef.GetLength());
- FX_DWORD uLableHash = FX_HashCode_String_GetW(CFX_WideString(wsLabelRef),
- wsLabelRef.GetLength());
for (int32_t i = 0; i < iCount; i++) {
CXFA_Object* refObj = rs.nodes[i];
if (!refObj->IsNode()) {
diff --git a/xfa/src/fxfa/src/app/xfa_fffield.cpp b/xfa/src/fxfa/src/app/xfa_fffield.cpp
index 80d65062a0..6e588b44c6 100644
--- a/xfa/src/fxfa/src/app/xfa_fffield.cpp
+++ b/xfa/src/fxfa/src/app/xfa_fffield.cpp
@@ -709,7 +709,6 @@ int32_t CXFA_FFField::CalculateOverride() {
if (!pAcc) {
return CalculateWidgetAcc(m_pDataAcc);
}
- int32_t iOverride = 0;
if (CalculateWidgetAcc(pAcc) == 0) {
return 0;
}
diff --git a/xfa/src/fxfa/src/app/xfa_fftextedit.cpp b/xfa/src/fxfa/src/app/xfa_fftextedit.cpp
index 60cb342b34..ce43ce0481 100644
--- a/xfa/src/fxfa/src/app/xfa_fftextedit.cpp
+++ b/xfa/src/fxfa/src/app/xfa_fftextedit.cpp
@@ -409,7 +409,6 @@ FWL_ERR CXFA_FFTextEdit::OnProcessEvent(CFWL_Event* pEvent) {
}
case FWL_EVTHASH_EDT_CheckWord: {
CFX_WideString wstr(L"FWL_EVENT_DTP_SelectChanged");
- FX_DWORD hashcode = FX_HashCode_String_GetW(wstr, wstr.GetLength());
CFWL_EvtEdtCheckWord* event = (CFWL_EvtEdtCheckWord*)pEvent;
event->bCheckWord = CheckWord(event->bsWord);
break;
diff --git a/xfa/src/fxfa/src/app/xfa_ffwidget.cpp b/xfa/src/fxfa/src/app/xfa_ffwidget.cpp
index ed35d3e0ba..cba6a7c459 100644
--- a/xfa/src/fxfa/src/app/xfa_ffwidget.cpp
+++ b/xfa/src/fxfa/src/app/xfa_ffwidget.cpp
@@ -1053,8 +1053,7 @@ CFX_DIBitmap* XFA_LoadImageFromBuffer(IFX_FileRead* pImageFileRead,
CFX_DIBitmap* pBitmap = NULL;
ICodec_ProgressiveDecoder* pProgressiveDecoder =
pCodecMgr->CreateProgressiveDecoder();
- FXCODEC_STATUS status =
- pProgressiveDecoder->LoadImageInfo(pImageFileRead, type, &dibAttr);
+ pProgressiveDecoder->LoadImageInfo(pImageFileRead, type, &dibAttr);
switch (dibAttr.m_wDPIUnit) {
case FXCODEC_RESUNIT_CENTIMETER:
dibAttr.m_nXDPI = (int32_t)(dibAttr.m_nXDPI * 2.54f);
diff --git a/xfa/src/fxfa/src/app/xfa_ffwidgetacc.cpp b/xfa/src/fxfa/src/app/xfa_ffwidgetacc.cpp
index 8a4c18823c..79f4ee1ccf 100644
--- a/xfa/src/fxfa/src/app/xfa_ffwidgetacc.cpp
+++ b/xfa/src/fxfa/src/app/xfa_ffwidgetacc.cpp
@@ -968,7 +968,6 @@ FX_BOOL CXFA_WidgetAcc::CalculateImageAutoSize(CFX_SizeF& size) {
}
size.Set(0, 0);
if (CFX_DIBitmap* pBitmap = GetImageImage()) {
- CXFA_Image imageObj = GetFormValue().GetImage();
CFX_RectF rtImage, rtFit;
rtImage.Set(0, 0, 0, 0);
rtFit.Set(0, 0, 0, 0);
diff --git a/xfa/src/fxfa/src/app/xfa_ffwidgethandler.cpp b/xfa/src/fxfa/src/app/xfa_ffwidgethandler.cpp
index ce2a91c3dc..4ef638333b 100644
--- a/xfa/src/fxfa/src/app/xfa_ffwidgethandler.cpp
+++ b/xfa/src/fxfa/src/app/xfa_ffwidgethandler.cpp
@@ -227,7 +227,7 @@ FX_BOOL CXFA_FFWidgetHandler::HasEvent(CXFA_WidgetAcc* pWidgetAcc,
if (!calc) {
return FALSE;
}
- if (CXFA_Script script = calc.GetScript()) {
+ if (calc.GetScript()) {
return TRUE;
}
return FALSE;
@@ -452,8 +452,7 @@ CXFA_Node* CXFA_FFWidgetHandler::CreateField(XFA_ELEMENT eElement,
CXFA_Node* pParent,
CXFA_Node* pBefore) const {
CXFA_Node* pField = CreateFormItem(XFA_ELEMENT_Field, pParent, pBefore);
- CXFA_Node* pUi = CreateCopyNode(XFA_ELEMENT_Ui, pField);
- CXFA_Node* pCheckButton = CreateCopyNode(eElement, pUi);
+ CreateCopyNode(eElement, CreateCopyNode(XFA_ELEMENT_Ui, pField));
CreateFontNode(pField);
return pField;
}
@@ -470,8 +469,7 @@ CXFA_Node* CXFA_FFWidgetHandler::CreateRectangle(CXFA_Node* pParent,
CXFA_Node* CXFA_FFWidgetHandler::CreateImage(CXFA_Node* pParent,
CXFA_Node* pBefore) const {
CXFA_Node* pField = CreateDraw(XFA_ELEMENT_Image, pParent, pBefore);
- CXFA_Node* pUi = CreateCopyNode(XFA_ELEMENT_Ui, pField);
- CXFA_Node* pImage = CreateCopyNode(XFA_ELEMENT_ImageEdit, pUi);
+ CreateCopyNode(XFA_ELEMENT_ImageEdit, CreateCopyNode(XFA_ELEMENT_Ui, pField));
return pField;
}
CXFA_Node* CXFA_FFWidgetHandler::CreateLine(CXFA_Node* pParent,
@@ -482,8 +480,7 @@ CXFA_Node* CXFA_FFWidgetHandler::CreateLine(CXFA_Node* pParent,
CXFA_Node* CXFA_FFWidgetHandler::CreateText(CXFA_Node* pParent,
CXFA_Node* pBefore) const {
CXFA_Node* pField = CreateDraw(XFA_ELEMENT_Text, pParent, pBefore);
- CXFA_Node* pUi = CreateCopyNode(XFA_ELEMENT_Ui, pField);
- CXFA_Node* pTextEdit = CreateCopyNode(XFA_ELEMENT_TextEdit, pUi);
+ CreateCopyNode(XFA_ELEMENT_TextEdit, CreateCopyNode(XFA_ELEMENT_Ui, pField));
CreateFontNode(pField);
return pField;
}
@@ -565,7 +562,7 @@ CXFA_Node* CXFA_FFWidgetHandler::CreateMarginNode(CXFA_Node* pParent,
CXFA_Node* CXFA_FFWidgetHandler::CreateValueNode(XFA_ELEMENT eValue,
CXFA_Node* pParent) const {
CXFA_Node* pValue = CreateCopyNode(XFA_ELEMENT_Value, pParent);
- CXFA_Node* pDate = CreateCopyNode(eValue, pValue);
+ CreateCopyNode(eValue, pValue);
return pValue;
}
IXFA_ObjFactory* CXFA_FFWidgetHandler::GetObjFactory() const {
diff --git a/xfa/src/fxfa/src/app/xfa_textlayout.cpp b/xfa/src/fxfa/src/app/xfa_textlayout.cpp
index 145da83baf..06f6b4cff9 100644
--- a/xfa/src/fxfa/src/app/xfa_textlayout.cpp
+++ b/xfa/src/fxfa/src/app/xfa_textlayout.cpp
@@ -1572,7 +1572,7 @@ FX_BOOL CXFA_TextLayout::LoadRichText(IFDE_XMLNode* pXMLNode,
}
if (wsName == FX_WSTRC(L"a")) {
if (pLinkData != NULL) {
- FX_DWORD dwRefCount = pLinkData->Release();
+ pLinkData->Release();
pLinkData = NULL;
}
}
diff --git a/xfa/src/fxfa/src/common/xfa_utils.h b/xfa/src/fxfa/src/common/xfa_utils.h
index bd3236967b..8f8a36f621 100644
--- a/xfa/src/fxfa/src/common/xfa_utils.h
+++ b/xfa/src/fxfa/src/common/xfa_utils.h
@@ -59,7 +59,6 @@ class CXFA_NodeIteratorTemplate {
}
NodeType* GetRoot() { return m_pRoot; }
NodeType* MoveToPrev() {
- NodeType* pCurrent = GetCurrent();
int32_t nStackLength = m_NodeStack.GetSize();
if (nStackLength == 1) {
return NULL;
diff --git a/xfa/src/fxfa/src/fm2js/xfa_fm2jscontext.cpp b/xfa/src/fxfa/src/fm2js/xfa_fm2jscontext.cpp
index 8641cae71b..8a87763bf2 100644
--- a/xfa/src/fxfa/src/fm2js/xfa_fm2jscontext.cpp
+++ b/xfa/src/fxfa/src/fm2js/xfa_fm2jscontext.cpp
@@ -325,7 +325,6 @@ void CXFA_FM2JSContext::Ceil(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
if (args.GetLength() == 1) {
FXJSE_HVALUE argValue = GetSimpleHValue(hThis, args, 0);
if (HValueIsNull(hThis, argValue)) {
@@ -409,7 +408,6 @@ void CXFA_FM2JSContext::Floor(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
if (args.GetLength() == 1) {
FXJSE_HVALUE argValue = GetSimpleHValue(hThis, args, 0);
if (HValueIsNull(hThis, argValue)) {
@@ -1347,8 +1345,8 @@ void CXFA_FM2JSContext::Num2Date(FXJSE_HOBJECT hThis,
CFX_ByteString szIsoDateString;
szIsoDateString.Format("%d%02d%02d", iYear + i, iMonth, iDay);
CFX_ByteString szLocalDateString;
- FX_BOOL bRet = IsoDate2Local(hThis, szIsoDateString, formatString,
- localString, szLocalDateString);
+ IsoDate2Local(hThis, szIsoDateString, formatString,
+ localString, szLocalDateString);
if (szLocalDateString.IsEmpty()) {
szLocalDateString = FX_BSTRC("");
}
@@ -3197,7 +3195,6 @@ void CXFA_FM2JSContext::Within(FXJSE_HOBJECT hThis,
void CXFA_FM2JSContext::If(FXJSE_HOBJECT hThis,
const CFX_ByteStringC& szFuncName,
CFXJSE_Arguments& args) {
- int32_t argc = args.GetLength();
if (args.GetLength() == 3) {
FXJSE_HVALUE argCondition = GetSimpleHValue(hThis, args, 0);
FXJSE_HVALUE argFirstValue = GetSimpleHValue(hThis, args, 1);
@@ -3463,10 +3460,8 @@ void CXFA_FM2JSContext::UnitValue(FXJSE_HOBJECT hThis,
while (*(pData + u) == 0x20 || *(pData + u) == 0x09 ||
*(pData + u) == 0x0B || *(pData + u) == 0x0C ||
*(pData + u) == 0x0A || *(pData + u) == 0x0D) {
- FX_CHAR ch = *(pData + u);
++u;
}
- int32_t uStart = u;
while (u < unitspanString.GetLength()) {
if ((*(pData + u) > '9' || *(pData + u) < '0') &&
*(pData + u) != '.' && *(pData + u) != '-') {
@@ -3611,7 +3606,6 @@ void CXFA_FM2JSContext::At(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
if (args.GetLength() == 2) {
FXJSE_HVALUE argOne = GetSimpleHValue(hThis, args, 0);
FXJSE_HVALUE argTwo = GetSimpleHValue(hThis, args, 1);
@@ -3641,7 +3635,6 @@ void CXFA_FM2JSContext::Concat(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
int32_t argc = args.GetLength();
if (argc >= 1) {
CFX_ByteString resultString;
@@ -3675,7 +3668,6 @@ void CXFA_FM2JSContext::Decode(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
int32_t argc = args.GetLength();
if (argc == 1) {
FXJSE_HVALUE argOne = GetSimpleHValue(hThis, args, 0);
@@ -3946,7 +3938,6 @@ void CXFA_FM2JSContext::Encode(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
int32_t argc = args.GetLength();
if (argc == 1) {
FXJSE_HVALUE argOne = GetSimpleHValue(hThis, args, 0);
@@ -4447,7 +4438,6 @@ void CXFA_FM2JSContext::Left(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
if (args.GetLength() == 2) {
FXJSE_HVALUE argOne = GetSimpleHValue(hThis, args, 0);
FXJSE_HVALUE argTwo = GetSimpleHValue(hThis, args, 1);
@@ -4479,7 +4469,6 @@ void CXFA_FM2JSContext::Len(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
if (args.GetLength() == 1) {
FXJSE_HVALUE argOne = GetSimpleHValue(hThis, args, 0);
if (HValueIsNull(hThis, argOne)) {
@@ -4550,7 +4539,6 @@ void CXFA_FM2JSContext::Ltrim(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
if (args.GetLength() == 1) {
FXJSE_HVALUE argOne = GetSimpleHValue(hThis, args, 0);
if (HValueIsNull(hThis, argOne)) {
@@ -4779,7 +4767,6 @@ void CXFA_FM2JSContext::Right(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
if (args.GetLength() == 2) {
FXJSE_HVALUE argOne = GetSimpleHValue(hThis, args, 0);
FXJSE_HVALUE argTwo = GetSimpleHValue(hThis, args, 1);
@@ -4811,7 +4798,6 @@ void CXFA_FM2JSContext::Rtrim(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
if (args.GetLength() == 1) {
FXJSE_HVALUE argOne = GetSimpleHValue(hThis, args, 0);
if (HValueIsNull(hThis, argOne)) {
@@ -4833,7 +4819,6 @@ void CXFA_FM2JSContext::Space(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
if (args.GetLength() == 1) {
FXJSE_HVALUE argOne = GetSimpleHValue(hThis, args, 0);
if (FXJSE_Value_IsNull(argOne)) {
@@ -6360,7 +6345,6 @@ void CXFA_FM2JSContext::eval_translation(FXJSE_HOBJECT hThis,
CFXJSE_Arguments& args) {
CXFA_FM2JSContext* pContext =
(CXFA_FM2JSContext*)FXJSE_Value_ToObject(hThis, NULL);
- FXJSE_HRUNTIME hruntime = pContext->GetScriptRuntime();
int32_t argc = args.GetLength();
if (argc == 1) {
FXJSE_HVALUE argOne = GetSimpleHValue(hThis, args, 0);
diff --git a/xfa/src/fxfa/src/parser/xfa_document_datamerger_imp.cpp b/xfa/src/fxfa/src/parser/xfa_document_datamerger_imp.cpp
index 8da91847fc..93ec1da24e 100644
--- a/xfa/src/fxfa/src/parser/xfa_document_datamerger_imp.cpp
+++ b/xfa/src/fxfa/src/parser/xfa_document_datamerger_imp.cpp
@@ -51,7 +51,7 @@ static void XFA_DataMerge_FormValueNode_MatchNoneCreateChild(
CXFA_Node* pFormNode) {
CXFA_WidgetData* pWidgetData = pFormNode->GetWidgetData();
FXSYS_assert(pWidgetData);
- XFA_ELEMENT eUIType = pWidgetData->GetUIType();
+ pWidgetData->GetUIType();
}
static FX_BOOL XFA_DataMerge_FormValueNode_SetChildContent(
CXFA_Node* pValueNode,
@@ -840,7 +840,7 @@ static CXFA_Node* XFA_DataMerge_CopyContainer_SubformSet(
subformMapArray.SetAt(pSubformNode, pDataNode);
subformArray.Add(pSubformNode);
}
- FX_POSITION position = subformMapArray.GetStartPosition();
+ subformMapArray.GetStartPosition();
for (int32_t iIndex = 0; iIndex < subformArray.GetSize(); iIndex++) {
CXFA_Node* pSubform = subformArray[iIndex];
CXFA_Node* pDataNode = (CXFA_Node*)subformMapArray.GetValueAt(pSubform);
@@ -862,7 +862,6 @@ static CXFA_Node* XFA_DataMerge_CopyContainer_SubformSet(
}
for (; iMax < 0 || iCurRepeatIndex < iMax; iCurRepeatIndex++) {
FX_BOOL bSelfMatch = FALSE;
- CXFA_Node* pOldNode = sNodeIterator.GetCurrent();
XFA_ATTRIBUTEENUM eBindMatch = XFA_ATTRIBUTEENUM_None;
if (!XFA_DataMerge_FindMatchingDataNode(
pDocument, pTemplateNode, pDataScope, bAccessedDataDOM, FALSE,
diff --git a/xfa/src/fxfa/src/parser/xfa_layout_itemlayout.cpp b/xfa/src/fxfa/src/parser/xfa_layout_itemlayout.cpp
index 22f8941b18..54279a53d0 100644
--- a/xfa/src/fxfa/src/parser/xfa_layout_itemlayout.cpp
+++ b/xfa/src/fxfa/src/parser/xfa_layout_itemlayout.cpp
@@ -1562,9 +1562,6 @@ static void XFA_ItemLayoutProcessor_AddTrailerBeforeSplit(
fBottomInset =
pMarginNode->GetMeasure(XFA_ATTRIBUTE_BottomInset).ToUnit(XFA_UNIT_Pt);
}
- FX_FLOAT fWidth = pTrailerLayoutItem->m_sSize.x;
- XFA_ATTRIBUTEENUM eLayout =
- pProcessor->m_pFormNode->GetEnum(XFA_ATTRIBUTE_Layout);
if (!pProcessor->IsAddNewRowForTrailer(pTrailerLayoutItem)) {
pTrailerLayoutItem->m_sPos.y = pProcessor->m_fLastRowY;
pTrailerLayoutItem->m_sPos.x = pProcessor->m_fLastRowWidth;
diff --git a/xfa/src/fxfa/src/parser/xfa_layout_pagemgr_new.cpp b/xfa/src/fxfa/src/parser/xfa_layout_pagemgr_new.cpp
index 4ab1dc96f2..5688d69e48 100644
--- a/xfa/src/fxfa/src/parser/xfa_layout_pagemgr_new.cpp
+++ b/xfa/src/fxfa/src/parser/xfa_layout_pagemgr_new.cpp
@@ -185,7 +185,6 @@ static void XFA_LayoutItemMgr_ReorderLayoutItemToTail(
if (!pParentLayoutItem) {
return;
}
- CXFA_LayoutItemImpl* pNext = pLayoutItem->m_pNextSibling;
pParentLayoutItem->RemoveChild(pLayoutItem);
pParentLayoutItem->AddChild(pLayoutItem);
}
@@ -196,7 +195,6 @@ static void XFA_LayoutItemMgr_RemoveLayoutItem(
if (!pParentLayoutItem) {
return;
}
- CXFA_LayoutItemImpl* pNext = pLayoutItem->m_pNextSibling;
pParentLayoutItem->RemoveChild(pLayoutItem);
}
void CXFA_LayoutPageMgr::RemoveLayoutRecord(CXFA_ContainerRecord* pNewRecord,
@@ -989,7 +987,6 @@ FX_BOOL CXFA_LayoutPageMgr::FindPageAreaFromPageSet(
if (pPageSet == NULL && pStartChild == NULL) {
return FALSE;
}
- int32_t iPageSetCount = 0;
if (IsPageSetRootOrderedOccurrence()) {
return FindPageAreaFromPageSet_Ordered(pPageSet, pStartChild,
pTargetPageArea, pTargetContentArea,
@@ -1080,9 +1077,6 @@ FX_BOOL CXFA_LayoutPageMgr::FindPageAreaFromPageSet_SimplexDuplex(
FX_BOOL bNewPage,
FX_BOOL bQuery,
XFA_ATTRIBUTEENUM ePreferredPosition) {
- XFA_ATTRIBUTEENUM eAdvisedPagePosition =
- pTargetPageArea ? pTargetPageArea->GetEnum(XFA_ATTRIBUTE_PagePosition)
- : (XFA_ATTRIBUTEENUM)-1;
const XFA_ATTRIBUTEENUM eFallbackPosition = XFA_ATTRIBUTEENUM_Any;
CXFA_Node *pPreferredPageArea = NULL, *pFallbackPageArea = NULL;
CXFA_Node* pCurrentNode = NULL;
diff --git a/xfa/src/fxfa/src/parser/xfa_object_imp.cpp b/xfa/src/fxfa/src/parser/xfa_object_imp.cpp
index 0bacfcd074..cf906072f3 100644
--- a/xfa/src/fxfa/src/parser/xfa_object_imp.cpp
+++ b/xfa/src/fxfa/src/parser/xfa_object_imp.cpp
@@ -693,8 +693,8 @@ void CXFA_Node::Script_Som_ResolveNodeList(FXJSE_HVALUE hValue,
if (refNode == NULL) {
refNode = this;
}
- int32_t iRet = pScriptContext->ResolveObjects(refNode, wsExpression,
- resoveNodeRS, dwFlag);
+ pScriptContext->ResolveObjects(refNode, wsExpression,
+ resoveNodeRS, dwFlag);
CXFA_ArrayNodeList* pNodeList = new CXFA_ArrayNodeList(m_pDocument);
if (resoveNodeRS.dwFlags == XFA_RESOVENODE_RSTYPE_Nodes) {
for (int32_t i = 0; i < resoveNodeRS.nodes.GetSize(); i++) {
@@ -2762,9 +2762,7 @@ void CXFA_Node::Script_Template_CreateNode(CFXJSE_Arguments* pArguments) {
}
}
void CXFA_Node::Script_Template_Recalculate(CFXJSE_Arguments* pArguments) {
- int32_t argc = pArguments->GetLength();
- if (argc == 1) {
- FX_BOOL bScriptFlags = pArguments->GetInt32(0) == 0 ? FALSE : TRUE;
+ if (pArguments->GetLength() == 1) {
FXJSE_Value_SetBoolean(pArguments->GetReturnValue(), TRUE);
} else {
ThrowScriptErrorMessage(XFA_IDS_INCORRECT_NUMBER_OF_METHOD, L"recalculate");
@@ -3336,8 +3334,6 @@ int32_t CXFA_Node::InstanceManager_SetInstances(int32_t iDesired) {
}
}
} else if (iDesired > iCount) {
- CXFA_Node* pTemplateNode = GetTemplateNode();
- CXFA_Node* pFormNode = GetNodeItem(XFA_NODEITEM_NextSibling);
while (iCount < iDesired) {
CXFA_Node* pNewInstance =
XFA_ScriptInstanceManager_CreateInstance(this, TRUE);
@@ -3416,7 +3412,6 @@ void CXFA_Node::Script_Form_FormNodes(CFXJSE_Arguments* pArguments) {
CXFA_Node* pDataNode = (CXFA_Node*)pArguments->GetObject(0);
if (pDataNode) {
CXFA_NodeArray formItems;
- int32_t iSize = pDataNode->GetBindItems(formItems);
CXFA_ArrayNodeList* pFormNodes = new CXFA_ArrayNodeList(m_pDocument);
pFormNodes->SetArrayNodeList(formItems);
FXJSE_Value_SetObject(
diff --git a/xfa/src/fxfa/src/parser/xfa_objectacc_imp.cpp b/xfa/src/fxfa/src/parser/xfa_objectacc_imp.cpp
index c239ac1d70..c7cd0f2078 100644
--- a/xfa/src/fxfa/src/parser/xfa_objectacc_imp.cpp
+++ b/xfa/src/fxfa/src/parser/xfa_objectacc_imp.cpp
@@ -2179,7 +2179,6 @@ int32_t CXFA_WidgetData::CountSelectedItems() {
int32_t CXFA_WidgetData::GetSelectedItem(int32_t nIndex) {
CFX_WideStringArray wsValueArray;
GetSelectedItemsValue(wsValueArray);
- int32_t iSelected = 0;
CFX_WideStringArray wsSaveTextArray;
GetChoiceListItems(wsSaveTextArray, TRUE);
int32_t iSaves = wsSaveTextArray.GetSize();
@@ -2822,7 +2821,6 @@ static CFX_WideString XFA_NumericNormalize(const CFX_WideString& wsValue,
int32_t iIndex = 0;
CFX_WideString wsRet;
FX_WCHAR* pRetBuffer = wsRet.GetBuffer(iCount);
- int32_t iDestCount = 0;
int32_t i = 0;
if (wsNewValue[i] == L'-') {
pRetBuffer[iIndex++] = '-';
diff --git a/xfa/src/fxfa/src/parser/xfa_parser_imp.cpp b/xfa/src/fxfa/src/parser/xfa_parser_imp.cpp
index 48547d7e9f..b0bbf4db75 100644
--- a/xfa/src/fxfa/src/parser/xfa_parser_imp.cpp
+++ b/xfa/src/fxfa/src/parser/xfa_parser_imp.cpp
@@ -421,8 +421,7 @@ CXFA_Node* CXFA_SimpleParser::ParseAsXDPPacket_XDP(
pPacketInfo->eFlags)) {
continue;
}
- if (CXFA_Node* pChildNode =
- pXFARootNode->GetFirstChildByName(pPacketInfo->uHash)) {
+ if (pXFARootNode->GetFirstChildByName(pPacketInfo->uHash)) {
return nullptr;
}
pXMLConfigDOMRoot = pChildItem;
@@ -1266,7 +1265,6 @@ void CXFA_SimpleParser::ParseDataValue(CXFA_Node* pXFANode,
continue;
}
CFX_WideString wsText;
- FX_BOOL bBreak = FALSE;
if (eNodeType == FDE_XMLNODE_Text) {
((IFDE_XMLText*)pXMLChild)->GetText(wsText);
if (!pXMLCurValueNode) {
@@ -1356,7 +1354,6 @@ void CXFA_SimpleParser::ParseInstruction(CXFA_Node* pXFANode,
}
CFX_WideString wsTargetName;
pXMLInstruction->GetTargetName(wsTargetName);
- int32_t iDataCount = pXMLInstruction->CountData();
if (wsTargetName == FX_WSTRC(L"originalXFAVersion")) {
CFX_WideString wsData;
if (pXMLInstruction->GetData(0, wsData) &&
diff --git a/xfa/src/fxfa/src/parser/xfa_script_hostpseudomodel.cpp b/xfa/src/fxfa/src/parser/xfa_script_hostpseudomodel.cpp
index 7b557f139a..21a4fa1997 100644
--- a/xfa/src/fxfa/src/parser/xfa_script_hostpseudomodel.cpp
+++ b/xfa/src/fxfa/src/parser/xfa_script_hostpseudomodel.cpp
@@ -506,7 +506,6 @@ void CScript_HostPseudoModel::Script_HostPseudoModel_SetFocus(
if (!pNotify) {
return;
}
- IXFA_Doc* hDoc = pNotify->GetHDOC();
CXFA_Node* pNode = NULL;
if (iLength >= 1) {
FXJSE_HVALUE hValue = pArguments->GetValue(0);
diff --git a/xfa/src/fxfa/src/parser/xfa_script_layoutpseudomodel.cpp b/xfa/src/fxfa/src/parser/xfa_script_layoutpseudomodel.cpp
index 14c8f2d409..554fde9363 100644
--- a/xfa/src/fxfa/src/parser/xfa_script_layoutpseudomodel.cpp
+++ b/xfa/src/fxfa/src/parser/xfa_script_layoutpseudomodel.cpp
@@ -32,7 +32,6 @@ void CScript_LayoutPseudoModel::Script_LayoutPseudoModel_Ready(
if (!pNotify) {
return;
}
- IXFA_Doc* hDoc = pNotify->GetHDOC();
if (bSetting) {
ThrowScriptErrorMessage(XFA_IDS_UNABLE_SET_READY);
return;
diff --git a/xfa/src/fxfa/src/parser/xfa_script_resolveprocessor.cpp b/xfa/src/fxfa/src/parser/xfa_script_resolveprocessor.cpp
index 7919395a56..befff73141 100644
--- a/xfa/src/fxfa/src/parser/xfa_script_resolveprocessor.cpp
+++ b/xfa/src/fxfa/src/parser/xfa_script_resolveprocessor.cpp
@@ -164,7 +164,6 @@ int32_t CXFA_ResolveProcessor::XFA_ResolveNodes_Excalmatory(
}
int32_t CXFA_ResolveProcessor::XFA_ResolveNodes_NumberSign(
CXFA_ResolveNodesData& rnd) {
- FX_DWORD dwStyles = rnd.m_dwStyles;
CFX_WideString wsName = rnd.m_wsName.Right(rnd.m_wsName.GetLength() - 1);
CFX_WideString wsCondition = rnd.m_wsCondition;
CXFA_Node* curNode = (CXFA_Node*)rnd.m_CurNode;
@@ -336,12 +335,10 @@ int32_t CXFA_ResolveProcessor::XFA_ResolveNodes_Normal(
}
}
if (dwStyles & XFA_RESOLVENODE_Properties) {
- FX_BOOL bSetFlag = FALSE;
for (int32_t i = 0; i < properties.GetSize(); i++) {
CXFA_Node* childProperty = properties[i];
if (childProperty->IsUnnamed()) {
uint32_t uPropHash = childProperty->GetClassHashCode();
- XFA_ELEMENT eName = childProperty->GetClassID();
if (uPropHash == uNameHash) {
nodes.Add(childProperty);
}
@@ -535,7 +532,6 @@ int32_t CXFA_ResolveProcessor::XFA_ResolveNodes_GetFilter(
CFX_WideString& wsCondition = rnd.m_wsCondition;
FX_WCHAR* pNameBuf = wsName.GetBuffer(iLength - nStart);
FX_WCHAR* pConditionBuf = wsCondition.GetBuffer(iLength - nStart);
- int32_t nCount = 0;
int32_t nNameCount = 0;
int32_t nConditionCount = 0;
CFX_Int32Array stack;
diff --git a/xfa/src/fxfa/src/parser/xfa_utils_imp.cpp b/xfa/src/fxfa/src/parser/xfa_utils_imp.cpp
index 1e7d9be1b4..abe5fdbd48 100644
--- a/xfa/src/fxfa/src/parser/xfa_utils_imp.cpp
+++ b/xfa/src/fxfa/src/parser/xfa_utils_imp.cpp
@@ -315,7 +315,6 @@ FX_DOUBLE XFA_WideStringToDouble(const CFX_WideString& wsStringVal) {
CFX_WideString wsValue = wsStringVal;
wsValue.TrimLeft();
wsValue.TrimRight();
- FX_BOOL bValid = TRUE;
int64_t nIntegral = 0;
FX_DWORD dwFractional = 0;
int32_t nExponent = 0;