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

Side by Side Diff: fpdfsdk/fpdfxfa/fpdfxfa_page.cpp

Issue 1821423002: Re-enable MSVC warning 4702 (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: address comments Created 4 years, 9 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
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 6
7 #include "fpdfsdk/include/fpdfxfa/fpdfxfa_page.h" 7 #include "fpdfsdk/include/fpdfxfa/fpdfxfa_page.h"
8 8
9 #include "core/fpdfapi/fpdf_page/include/cpdf_page.h" 9 #include "core/fpdfapi/fpdf_page/include/cpdf_page.h"
10 #include "core/fpdfapi/fpdf_parser/include/cpdf_document.h" 10 #include "core/fpdfapi/fpdf_parser/include/cpdf_document.h"
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 case DOCTYPE_PDF: 95 case DOCTYPE_PDF:
96 case DOCTYPE_STATIC_XFA: { 96 case DOCTYPE_STATIC_XFA: {
97 return LoadPDFPage(); 97 return LoadPDFPage();
98 } 98 }
99 case DOCTYPE_DYNAMIC_XFA: { 99 case DOCTYPE_DYNAMIC_XFA: {
100 return LoadXFAPageView(); 100 return LoadXFAPageView();
101 } 101 }
102 default: 102 default:
103 return FALSE; 103 return FALSE;
104 } 104 }
105
106 return FALSE;
107 } 105 }
108 106
109 FX_BOOL CPDFXFA_Page::LoadPDFPage(CPDF_Dictionary* pageDict) { 107 FX_BOOL CPDFXFA_Page::LoadPDFPage(CPDF_Dictionary* pageDict) {
110 if (!m_pDocument || m_iPageIndex < 0 || !pageDict) 108 if (!m_pDocument || m_iPageIndex < 0 || !pageDict)
111 return FALSE; 109 return FALSE;
112 110
113 if (m_pPDFPage) 111 if (m_pPDFPage)
114 delete m_pPDFPage; 112 delete m_pPDFPage;
115 113
116 m_pPDFPage = new CPDF_Page(); 114 m_pPDFPage = new CPDF_Page();
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
241 case DOCTYPE_PDF: 239 case DOCTYPE_PDF:
242 case DOCTYPE_STATIC_XFA: { 240 case DOCTYPE_STATIC_XFA: {
243 if (m_pPDFPage) { 241 if (m_pPDFPage) {
244 m_pPDFPage->GetDisplayMatrix(matrix, xPos, yPos, xSize, ySize, iRotate); 242 m_pPDFPage->GetDisplayMatrix(matrix, xPos, yPos, xSize, ySize, iRotate);
245 } 243 }
246 } break; 244 } break;
247 default: 245 default:
248 return; 246 return;
249 } 247 }
250 } 248 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698