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

Side by Side Diff: xfa/src/fxbarcode/qrcode/BC_QRDetector.cpp

Issue 1577503002: Merge to XFA: Switch most min/max macros to std::min/max. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@xfa
Patch Set: rebase Created 4 years, 11 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 unified diff | Download patch
« no previous file with comments | « xfa/src/fxbarcode/qrcode/BC_QRCoderEncoder.cpp ('k') | xfa/src/fxfa/src/app/xfa_ffapp.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 PDFium Authors. All rights reserved. 1 // Copyright 2014 PDFium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com 5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
6 // Original code is licensed as follows: 6 // Original code is licensed as follows:
7 /* 7 /*
8 * Copyright 2007 ZXing authors 8 * Copyright 2007 ZXing authors
9 * 9 *
10 * Licensed under the Apache License, Version 2.0 (the "License"); 10 * Licensed under the Apache License, Version 2.0 (the "License");
11 * you may not use this file except in compliance with the License. 11 * you may not use this file except in compliance with the License.
12 * You may obtain a copy of the License at 12 * You may obtain a copy of the License at
13 * 13 *
14 * http://www.apache.org/licenses/LICENSE-2.0 14 * http://www.apache.org/licenses/LICENSE-2.0
15 * 15 *
16 * Unless required by applicable law or agreed to in writing, software 16 * Unless required by applicable law or agreed to in writing, software
17 * distributed under the License is distributed on an "AS IS" BASIS, 17 * distributed under the License is distributed on an "AS IS" BASIS,
18 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. 18 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
19 * See the License for the specific language governing permissions and 19 * See the License for the specific language governing permissions and
20 * limitations under the License. 20 * limitations under the License.
21 */ 21 */
22 22
23 #include <algorithm>
24
23 #include "xfa/src/fxbarcode/barcode.h" 25 #include "xfa/src/fxbarcode/barcode.h"
24 #include "xfa/src/fxbarcode/common/BC_CommonBitMatrix.h" 26 #include "xfa/src/fxbarcode/common/BC_CommonBitMatrix.h"
25 #include "xfa/src/fxbarcode/BC_ResultPoint.h" 27 #include "xfa/src/fxbarcode/BC_ResultPoint.h"
26 #include "BC_QRFinderPattern.h" 28 #include "BC_QRFinderPattern.h"
27 #include "BC_QRCoderVersion.h" 29 #include "BC_QRCoderVersion.h"
28 #include "BC_FinderPatternInfo.h" 30 #include "BC_FinderPatternInfo.h"
29 #include "BC_QRGridSampler.h" 31 #include "BC_QRGridSampler.h"
30 #include "BC_QRAlignmentPatternFinder.h" 32 #include "BC_QRAlignmentPatternFinder.h"
31 #include "BC_QRFinderPatternFinder.h" 33 #include "BC_QRFinderPatternFinder.h"
32 #include "BC_QRDetectorResult.h" 34 #include "BC_QRDetectorResult.h"
(...skipping 215 matching lines...) Expand 10 before | Expand all | Expand 10 after
248 int32_t diffY = toY - fromY; 250 int32_t diffY = toY - fromY;
249 return (FX_FLOAT)sqrt((double)(diffX * diffX + diffY * diffY)); 251 return (FX_FLOAT)sqrt((double)(diffX * diffX + diffY * diffY));
250 } 252 }
251 CBC_QRAlignmentPattern* CBC_QRDetector::FindAlignmentInRegion( 253 CBC_QRAlignmentPattern* CBC_QRDetector::FindAlignmentInRegion(
252 FX_FLOAT overallEstModuleSize, 254 FX_FLOAT overallEstModuleSize,
253 int32_t estAlignmentX, 255 int32_t estAlignmentX,
254 int32_t estAlignmentY, 256 int32_t estAlignmentY,
255 FX_FLOAT allowanceFactor, 257 FX_FLOAT allowanceFactor,
256 int32_t& e) { 258 int32_t& e) {
257 int32_t allowance = (int32_t)(allowanceFactor * overallEstModuleSize); 259 int32_t allowance = (int32_t)(allowanceFactor * overallEstModuleSize);
258 int32_t alignmentAreaLeftX = FX_MAX(0, estAlignmentX - allowance); 260 int32_t alignmentAreaLeftX = std::max(0, estAlignmentX - allowance);
259 int32_t alignmentAreaRightX = 261 int32_t alignmentAreaRightX =
260 FX_MIN(m_image->GetWidth() - 1, estAlignmentX + allowance); 262 std::min(m_image->GetWidth() - 1, estAlignmentX + allowance);
261 if (alignmentAreaRightX - alignmentAreaLeftX < overallEstModuleSize * 3) { 263 if (alignmentAreaRightX - alignmentAreaLeftX < overallEstModuleSize * 3) {
262 e = BCExceptionRead; 264 e = BCExceptionRead;
263 BC_EXCEPTION_CHECK_ReturnValue(e, NULL); 265 BC_EXCEPTION_CHECK_ReturnValue(e, NULL);
264 } 266 }
265 int32_t alignmentAreaTopY = FX_MAX(0, estAlignmentY - allowance); 267 int32_t alignmentAreaTopY = std::max(0, estAlignmentY - allowance);
266 int32_t alignmentAreaBottomY = 268 int32_t alignmentAreaBottomY =
267 FX_MIN(m_image->GetHeight() - 1, estAlignmentY + allowance); 269 std::min(m_image->GetHeight() - 1, estAlignmentY + allowance);
268 CBC_QRAlignmentPatternFinder alignmentFinder( 270 CBC_QRAlignmentPatternFinder alignmentFinder(
269 m_image, alignmentAreaLeftX, alignmentAreaTopY, 271 m_image, alignmentAreaLeftX, alignmentAreaTopY,
270 alignmentAreaRightX - alignmentAreaLeftX, 272 alignmentAreaRightX - alignmentAreaLeftX,
271 alignmentAreaBottomY - alignmentAreaTopY, overallEstModuleSize); 273 alignmentAreaBottomY - alignmentAreaTopY, overallEstModuleSize);
272 CBC_QRAlignmentPattern* qap = alignmentFinder.Find(e); 274 CBC_QRAlignmentPattern* qap = alignmentFinder.Find(e);
273 BC_EXCEPTION_CHECK_ReturnValue(e, NULL); 275 BC_EXCEPTION_CHECK_ReturnValue(e, NULL);
274 return qap; 276 return qap;
275 } 277 }
OLDNEW
« no previous file with comments | « xfa/src/fxbarcode/qrcode/BC_QRCoderEncoder.cpp ('k') | xfa/src/fxfa/src/app/xfa_ffapp.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698