From c09625ca59701fabeb49dc59edcf33031b2c6672 Mon Sep 17 00:00:00 2001 From: tsepez Date: Mon, 7 Nov 2016 11:46:09 -0800 Subject: Rename CPDF_Linearized to CPDF_LinearizedHeader My OCD insists that classes be named after nouns, and "linearized" feels like an adjective. Remove a redundant "if" while at it. Review-Url: https://codereview.chromium.org/2482973002 --- core/fpdfapi/parser/cpdf_linearized.cpp | 71 --------------------------------- 1 file changed, 71 deletions(-) delete mode 100644 core/fpdfapi/parser/cpdf_linearized.cpp (limited to 'core/fpdfapi/parser/cpdf_linearized.cpp') diff --git a/core/fpdfapi/parser/cpdf_linearized.cpp b/core/fpdfapi/parser/cpdf_linearized.cpp deleted file mode 100644 index ec57f100cb..0000000000 --- a/core/fpdfapi/parser/cpdf_linearized.cpp +++ /dev/null @@ -1,71 +0,0 @@ -// Copyright 2016 PDFium Authors. All rights reserved. -// Use of this source code is governed by a BSD-style license that can be -// found in the LICENSE file. - -// Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com - -#include "core/fpdfapi/parser/cpdf_linearized.h" - -#include "core/fpdfapi/parser/cpdf_array.h" -#include "core/fpdfapi/parser/cpdf_dictionary.h" -#include "core/fpdfapi/parser/cpdf_number.h" -#include "third_party/base/ptr_util.h" - -namespace { - -template -bool IsValidNumericDictionaryValue(const CPDF_Dictionary* pDict, - const char* key, - T min_value, - bool must_exist = true) { - if (!pDict->KeyExist(key)) - return !must_exist; - const CPDF_Number* pNum = ToNumber(pDict->GetObjectFor(key)); - if (!pNum || !pNum->IsInteger()) - return false; - const int raw_value = pNum->GetInteger(); - if (!pdfium::base::IsValueInRangeForNumericType(raw_value)) - return false; - return static_cast(raw_value) >= min_value; -} - -} // namespace - -// static -std::unique_ptr CPDF_Linearized::CreateForObject( - std::unique_ptr pObj) { - auto pDict = ToDictionary(std::move(pObj)); - if (!pDict || !pDict->KeyExist("Linearized") || - !IsValidNumericDictionaryValue(pDict.get(), "L", 1) || - !IsValidNumericDictionaryValue(pDict.get(), "P", 0, false) || - !IsValidNumericDictionaryValue(pDict.get(), "T", 1) || - !IsValidNumericDictionaryValue(pDict.get(), "N", 0) || - !IsValidNumericDictionaryValue(pDict.get(), "E", 1) || - !IsValidNumericDictionaryValue(pDict.get(), "O", 1)) - return nullptr; - return pdfium::WrapUnique(new CPDF_Linearized(pDict.get())); -} - -CPDF_Linearized::CPDF_Linearized(const CPDF_Dictionary* pDict) { - if (!pDict) - return; - m_szFileSize = pDict->GetIntegerFor("L"); - m_dwFirstPageNo = pDict->GetIntegerFor("P"); - m_szLastXRefOffset = pDict->GetIntegerFor("T"); - m_PageCount = pDict->GetIntegerFor("N"); - m_szFirstPageEndOffset = pDict->GetIntegerFor("E"); - m_FirstPageObjNum = pDict->GetIntegerFor("O"); - const CPDF_Array* pHintStreamRange = pDict->GetArrayFor("H"); - const size_t nHintStreamSize = - pHintStreamRange ? pHintStreamRange->GetCount() : 0; - if (nHintStreamSize == 2 || nHintStreamSize == 4) { - m_szHintStart = std::max(pHintStreamRange->GetIntegerAt(0), 0); - m_szHintLength = std::max(pHintStreamRange->GetIntegerAt(1), 0); - } -} - -CPDF_Linearized::~CPDF_Linearized() {} - -bool CPDF_Linearized::HasHintTable() const { - return GetPageCount() > 1 && GetHintStart() > 0 && GetHintLength() > 0; -} -- cgit v1.2.3