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

Side by Side Diff: core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp

Issue 429593005: Fix a few more warnings in chromium_code mode. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 6 years, 4 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 "../../../include/fpdfapi/fpdf_page.h" 7 #include "../../../include/fpdfapi/fpdf_page.h"
8 #include "../../../include/fpdfapi/fpdf_module.h" 8 #include "../../../include/fpdfapi/fpdf_module.h"
9 #include "../../../include/fpdfapi/fpdf_serial.h" 9 #include "../../../include/fpdfapi/fpdf_serial.h"
10 #include "pageint.h" 10 #include "pageint.h"
(...skipping 601 matching lines...) Expand 10 before | Expand all | Expand 10 after
612 return; 612 return;
613 } 613 }
614 AddPathPoint(GetNumber(5), GetNumber(4), FXPT_BEZIERTO); 614 AddPathPoint(GetNumber(5), GetNumber(4), FXPT_BEZIERTO);
615 AddPathPoint(GetNumber(3), GetNumber(2), FXPT_BEZIERTO); 615 AddPathPoint(GetNumber(3), GetNumber(2), FXPT_BEZIERTO);
616 AddPathPoint(GetNumber(1), GetNumber(0), FXPT_BEZIERTO); 616 AddPathPoint(GetNumber(1), GetNumber(0), FXPT_BEZIERTO);
617 } 617 }
618 void CPDF_StreamContentParser::Handle_ConcatMatrix() 618 void CPDF_StreamContentParser::Handle_ConcatMatrix()
619 { 619 {
620 FX_FLOAT a2 = GetNumber16(5), b2 = GetNumber16(4), c2 = GetNumber16(3), d2 = GetNumber16(2); 620 FX_FLOAT a2 = GetNumber16(5), b2 = GetNumber16(4), c2 = GetNumber16(3), d2 = GetNumber16(2);
621 FX_FLOAT e2 = GetNumber(1), f2 = GetNumber(0); 621 FX_FLOAT e2 = GetNumber(1), f2 = GetNumber(0);
622 FX_FLOAT old_width_scale = m_pCurStates->m_CTM.GetXUnit();
623 CFX_AffineMatrix new_matrix(a2, b2, c2, d2, e2, f2); 622 CFX_AffineMatrix new_matrix(a2, b2, c2, d2, e2, f2);
624 new_matrix.Concat(m_pCurStates->m_CTM); 623 new_matrix.Concat(m_pCurStates->m_CTM);
625 m_pCurStates->m_CTM = new_matrix; 624 m_pCurStates->m_CTM = new_matrix;
626 FX_FLOAT new_width_scale = m_pCurStates->m_CTM.GetXUnit();
627 OnChangeTextMatrix(); 625 OnChangeTextMatrix();
628 } 626 }
629 void CPDF_StreamContentParser::Handle_SetColorSpace_Fill() 627 void CPDF_StreamContentParser::Handle_SetColorSpace_Fill()
630 { 628 {
631 if (m_Options.m_bTextOnly) { 629 if (m_Options.m_bTextOnly) {
632 return; 630 return;
633 } 631 }
634 CFX_ByteString csname = GetString(0); 632 CFX_ByteString csname = GetString(0);
635 CPDF_ColorSpace* pCS = FindColorSpace(csname); 633 CPDF_ColorSpace* pCS = FindColorSpace(csname);
636 if (pCS == NULL) { 634 if (pCS == NULL) {
(...skipping 1038 matching lines...) Expand 10 before | Expand all | Expand 10 after
1675 buf.AppendChar((char)code); 1673 buf.AppendChar((char)code);
1676 } 1674 }
1677 bFirst = !bFirst; 1675 bFirst = !bFirst;
1678 } 1676 }
1679 } 1677 }
1680 if (!bFirst) { 1678 if (!bFirst) {
1681 buf.AppendChar((char)code); 1679 buf.AppendChar((char)code);
1682 } 1680 }
1683 return buf.GetByteString(); 1681 return buf.GetByteString();
1684 } 1682 }
OLDNEW
« no previous file with comments | « core/src/fpdfapi/fpdf_page/fpdf_page_doc.cpp ('k') | core/src/fpdfapi/fpdf_parser/fpdf_parser_encrypt.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698