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

Side by Side Diff: xfa/fxbarcode/cbc_onecode.cpp

Issue 2037573005: Clean up C-Style casts in CBC_CodeBase and subclasses (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: 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 unified diff | Download patch
« no previous file with comments | « xfa/fxbarcode/cbc_onecode.h ('k') | xfa/fxbarcode/cbc_pdf417i.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 2016 PDFium Authors. All rights reserved. 1 // Copyright 2016 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 * Copyright 2011 ZXing authors 7 * Copyright 2011 ZXing authors
8 * 8 *
9 * Licensed under the Apache License, Version 2.0 (the "License"); 9 * Licensed under the Apache License, Version 2.0 (the "License");
10 * you may not use this file except in compliance with the License. 10 * you may not use this file except in compliance with the License.
11 * You may obtain a copy of the License at 11 * You may obtain a copy of the License at
12 * 12 *
13 * http://www.apache.org/licenses/LICENSE-2.0 13 * http://www.apache.org/licenses/LICENSE-2.0
14 * 14 *
15 * Unless required by applicable law or agreed to in writing, software 15 * Unless required by applicable law or agreed to in writing, software
16 * distributed under the License is distributed on an "AS IS" BASIS, 16 * distributed under the License is distributed on an "AS IS" BASIS,
17 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. 17 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
18 * See the License for the specific language governing permissions and 18 * See the License for the specific language governing permissions and
19 * limitations under the License. 19 * limitations under the License.
20 */ 20 */
21 21
22 #include "xfa/fxbarcode/cbc_onecode.h" 22 #include "xfa/fxbarcode/cbc_onecode.h"
23 23
24 #include "xfa/fxbarcode/oned/BC_OneDimWriter.h" 24 #include "xfa/fxbarcode/oned/BC_OneDimWriter.h"
25 25
26 CBC_OneCode::CBC_OneCode() {} 26 CBC_OneCode::CBC_OneCode(CBC_Reader* pReader, CBC_Writer* pWriter)
27 : CBC_CodeBase(pReader, pWriter) {}
27 28
28 CBC_OneCode::~CBC_OneCode() {} 29 CBC_OneCode::~CBC_OneCode() {}
29 30
30 FX_BOOL CBC_OneCode::CheckContentValidity(const CFX_WideStringC& contents) { 31 FX_BOOL CBC_OneCode::CheckContentValidity(const CFX_WideStringC& contents) {
31 if (m_pBCWriter) 32 return m_pBCWriter &&
32 return ((CBC_OneDimWriter*)m_pBCWriter)->CheckContentValidity(contents); 33 static_cast<CBC_OneDimWriter*>(m_pBCWriter.get())
33 return FALSE; 34 ->CheckContentValidity(contents);
34 } 35 }
35 36
36 CFX_WideString CBC_OneCode::FilterContents(const CFX_WideStringC& contents) { 37 CFX_WideString CBC_OneCode::FilterContents(const CFX_WideStringC& contents) {
37 CFX_WideString tmp;
38 if (!m_pBCWriter) 38 if (!m_pBCWriter)
39 return tmp; 39 return CFX_WideString();
40 return ((CBC_OneDimWriter*)m_pBCWriter)->FilterContents(contents); 40 return static_cast<CBC_OneDimWriter*>(m_pBCWriter.get())
41 ->FilterContents(contents);
41 } 42 }
42 43
43 void CBC_OneCode::SetPrintChecksum(FX_BOOL checksum) { 44 void CBC_OneCode::SetPrintChecksum(FX_BOOL checksum) {
44 if (m_pBCWriter) 45 if (m_pBCWriter)
45 ((CBC_OneDimWriter*)m_pBCWriter)->SetPrintChecksum(checksum); 46 static_cast<CBC_OneDimWriter*>(m_pBCWriter.get())
47 ->SetPrintChecksum(checksum);
46 } 48 }
47 49
48 void CBC_OneCode::SetDataLength(int32_t length) { 50 void CBC_OneCode::SetDataLength(int32_t length) {
49 if (m_pBCWriter) 51 if (m_pBCWriter)
50 ((CBC_OneDimWriter*)m_pBCWriter)->SetDataLength(length); 52 static_cast<CBC_OneDimWriter*>(m_pBCWriter.get())->SetDataLength(length);
51 } 53 }
52 54
53 void CBC_OneCode::SetCalChecksum(FX_BOOL calc) { 55 void CBC_OneCode::SetCalChecksum(FX_BOOL calc) {
54 if (m_pBCWriter) 56 if (m_pBCWriter)
55 ((CBC_OneDimWriter*)m_pBCWriter)->SetCalcChecksum(calc); 57 static_cast<CBC_OneDimWriter*>(m_pBCWriter.get())->SetCalcChecksum(calc);
56 } 58 }
57 59
58 FX_BOOL CBC_OneCode::SetFont(CFX_Font* cFont) { 60 FX_BOOL CBC_OneCode::SetFont(CFX_Font* cFont) {
59 if (m_pBCWriter) 61 if (m_pBCWriter)
60 return ((CBC_OneDimWriter*)m_pBCWriter)->SetFont(cFont); 62 return static_cast<CBC_OneDimWriter*>(m_pBCWriter.get())->SetFont(cFont);
61 return FALSE; 63 return FALSE;
62 } 64 }
63 65
64 void CBC_OneCode::SetFontSize(FX_FLOAT size) { 66 void CBC_OneCode::SetFontSize(FX_FLOAT size) {
65 if (m_pBCWriter) 67 if (m_pBCWriter)
66 ((CBC_OneDimWriter*)m_pBCWriter)->SetFontSize(size); 68 static_cast<CBC_OneDimWriter*>(m_pBCWriter.get())->SetFontSize(size);
67 } 69 }
68 70
69 void CBC_OneCode::SetFontStyle(int32_t style) { 71 void CBC_OneCode::SetFontStyle(int32_t style) {
70 if (m_pBCWriter) 72 if (m_pBCWriter)
71 ((CBC_OneDimWriter*)m_pBCWriter)->SetFontStyle(style); 73 static_cast<CBC_OneDimWriter*>(m_pBCWriter.get())->SetFontStyle(style);
72 } 74 }
73 75
74 void CBC_OneCode::SetFontColor(FX_ARGB color) { 76 void CBC_OneCode::SetFontColor(FX_ARGB color) {
75 if (m_pBCWriter) 77 if (m_pBCWriter)
76 ((CBC_OneDimWriter*)m_pBCWriter)->SetFontColor(color); 78 static_cast<CBC_OneDimWriter*>(m_pBCWriter.get())->SetFontColor(color);
77 } 79 }
OLDNEW
« no previous file with comments | « xfa/fxbarcode/cbc_onecode.h ('k') | xfa/fxbarcode/cbc_pdf417i.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698