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

Side by Side Diff: core/src/fxcodec/codec/fx_codec_icc.cpp

Issue 1297713003: Don't bother checking pointers before delete[] and FX_Free(). (Closed) Base URL: https://pdfium.googlesource.com/pdfium@master
Patch Set: rebase Created 5 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
« no previous file with comments | « core/src/fxcodec/codec/fx_codec_flate.cpp ('k') | core/src/fxcodec/codec/fx_codec_jbig.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 "../../../../third_party/lcms2-2.6/include/lcms2.h" 7 #include "../../../../third_party/lcms2-2.6/include/lcms2.h"
8 #include "../../../include/fxcodec/fx_codec.h" 8 #include "../../../include/fxcodec/fx_codec.h"
9 #include "codec_int.h" 9 #include "codec_int.h"
10 10
(...skipping 374 matching lines...) Expand 10 before | Expand all | Expand 10 after
385 }; 385 };
386 CFX_IccTransformCache::CFX_IccTransformCache(CLcmsCmm* pCmm) { 386 CFX_IccTransformCache::CFX_IccTransformCache(CLcmsCmm* pCmm) {
387 m_pIccTransform = NULL; 387 m_pIccTransform = NULL;
388 m_dwRate = 1; 388 m_dwRate = 1;
389 m_pCmm = pCmm; 389 m_pCmm = pCmm;
390 } 390 }
391 CFX_IccTransformCache::~CFX_IccTransformCache() { 391 CFX_IccTransformCache::~CFX_IccTransformCache() {
392 if (m_pIccTransform) { 392 if (m_pIccTransform) {
393 cmsDeleteTransform(m_pIccTransform); 393 cmsDeleteTransform(m_pIccTransform);
394 } 394 }
395 if (m_pCmm) { 395 FX_Free(m_pCmm);
396 FX_Free(m_pCmm);
397 }
398 } 396 }
399 void CFX_IccTransformCache::Purge() {} 397 void CFX_IccTransformCache::Purge() {}
400 class CFX_ByteStringKey : public CFX_BinaryBuf { 398 class CFX_ByteStringKey : public CFX_BinaryBuf {
401 public: 399 public:
402 CFX_ByteStringKey() : CFX_BinaryBuf() {} 400 CFX_ByteStringKey() : CFX_BinaryBuf() {}
403 CFX_ByteStringKey& operator<<(FX_DWORD i); 401 CFX_ByteStringKey& operator<<(FX_DWORD i);
404 }; 402 };
405 CFX_ByteStringKey& CFX_ByteStringKey::operator<<(FX_DWORD i) { 403 CFX_ByteStringKey& CFX_ByteStringKey::operator<<(FX_DWORD i) {
406 AppendBlock(&i, sizeof(FX_DWORD)); 404 AppendBlock(&i, sizeof(FX_DWORD));
407 return *this; 405 return *this;
(...skipping 1566 matching lines...) Expand 10 before | Expand all | Expand 10 after
1974 uint8_t c1 = FXSYS_round(c * 255); 1972 uint8_t c1 = FXSYS_round(c * 255);
1975 uint8_t m1 = FXSYS_round(m * 255); 1973 uint8_t m1 = FXSYS_round(m * 255);
1976 uint8_t y1 = FXSYS_round(y * 255); 1974 uint8_t y1 = FXSYS_round(y * 255);
1977 uint8_t k1 = FXSYS_round(k * 255); 1975 uint8_t k1 = FXSYS_round(k * 255);
1978 uint8_t r, g, b; 1976 uint8_t r, g, b;
1979 AdobeCMYK_to_sRGB1(c1, m1, y1, k1, r, g, b); 1977 AdobeCMYK_to_sRGB1(c1, m1, y1, k1, r, g, b);
1980 R = 1.0f * r / 255; 1978 R = 1.0f * r / 255;
1981 G = 1.0f * g / 255; 1979 G = 1.0f * g / 255;
1982 B = 1.0f * b / 255; 1980 B = 1.0f * b / 255;
1983 } 1981 }
OLDNEW
« no previous file with comments | « core/src/fxcodec/codec/fx_codec_flate.cpp ('k') | core/src/fxcodec/codec/fx_codec_jbig.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698