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

Side by Side Diff: core/src/fpdfdoc/doc_metadata.cpp

Issue 1243953004: Re-land else-after-returns (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 5 years, 5 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 | « core/src/fpdfdoc/doc_formfield.cpp ('k') | core/src/fpdfdoc/doc_ocg.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 6
7 #include "../../include/fpdfdoc/fpdf_doc.h" 7 #include "../../include/fpdfdoc/fpdf_doc.h"
8 #include "../../include/fxcrt/fx_xml.h" 8 #include "../../include/fxcrt/fx_xml.h"
9 typedef struct _PDFDOC_METADATA { 9 typedef struct _PDFDOC_METADATA {
10 CPDF_Document *m_pDoc; 10 CPDF_Document *m_pDoc;
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("Alt")); 105 pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("Alt"));
106 if (!pElmnt) { 106 if (!pElmnt) {
107 continue; 107 continue;
108 } 108 }
109 pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("li")); 109 pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("li"));
110 if (!pElmnt) { 110 if (!pElmnt) {
111 continue; 111 continue;
112 } 112 }
113 wsStr = pElmnt->GetContent(0); 113 wsStr = pElmnt->GetContent(0);
114 return wsStr.GetLength(); 114 return wsStr.GetLength();
115 } else if (bsItem == FX_BSTRC("Author")) { 115 }
116 if (bsItem == FX_BSTRC("Author")) {
116 CXML_Element *pElmnt = pTag->GetElement(NULL, bsTag); 117 CXML_Element *pElmnt = pTag->GetElement(NULL, bsTag);
117 if (!pElmnt) { 118 if (!pElmnt) {
118 continue; 119 continue;
119 } 120 }
120 pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("Seq")); 121 pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("Seq"));
121 if (!pElmnt) { 122 if (!pElmnt) {
122 continue; 123 continue;
123 } 124 }
124 pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("li")); 125 pElmnt = pElmnt->GetElement(NULL, FX_BSTRC("li"));
125 if (!pElmnt) { 126 if (!pElmnt) {
126 continue; 127 continue;
127 } 128 }
128 wsStr = pElmnt->GetContent(0); 129 wsStr = pElmnt->GetContent(0);
129 return wsStr.GetLength(); 130 return wsStr.GetLength();
130 } else {
131 CXML_Element *pElmnt = pTag->GetElement(NULL, bsTag);
132 if (!pElmnt) {
133 continue;
134 }
135 wsStr = pElmnt->GetContent(0);
136 return wsStr.GetLength();
137 } 131 }
132 CXML_Element *pElmnt = pTag->GetElement(NULL, bsTag);
133 if (!pElmnt) {
134 continue;
135 }
136 wsStr = pElmnt->GetContent(0);
137 return wsStr.GetLength();
138 } 138 }
139 return -1; 139 return -1;
140 } 140 }
141 CXML_Element* CPDF_Metadata::GetRoot() const 141 CXML_Element* CPDF_Metadata::GetRoot() const
142 { 142 {
143 return ((PDFDOC_LPMETADATA)m_pData)->m_pXmlElmnt; 143 return ((PDFDOC_LPMETADATA)m_pData)->m_pXmlElmnt;
144 } 144 }
145 CXML_Element* CPDF_Metadata::GetRDF() const 145 CXML_Element* CPDF_Metadata::GetRDF() const
146 { 146 {
147 return ((PDFDOC_LPMETADATA)m_pData)->m_pElmntRdf; 147 return ((PDFDOC_LPMETADATA)m_pData)->m_pElmntRdf;
148 } 148 }
OLDNEW
« no previous file with comments | « core/src/fpdfdoc/doc_formfield.cpp ('k') | core/src/fpdfdoc/doc_ocg.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698