diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-06-27 18:24:10 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-06-27 18:24:10 +0000 |
| commit | 8444b6a1bf03678c195e78612585e93724baacc5 (patch) | |
| tree | 2cbd0c48955e69532370b24eea9c9ea2885026fa /core/fxcodec/codec | |
| parent | 20eca1e383b7dce69cd791d42bda3558a3966301 (diff) | |
| download | pdfium-8444b6a1bf03678c195e78612585e93724baacc5.tar.xz | |
Remove unused m_bGif* fields from CFX_DIBAttribute
Two are completely unused, and two are set but never referenced.
Change-Id: I0173123e33a4240aef9ed24aaee9e1df6cf9aa0a
Reviewed-on: https://pdfium-review.googlesource.com/36290
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'core/fxcodec/codec')
| -rw-r--r-- | core/fxcodec/codec/ccodec_gifmodule.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/core/fxcodec/codec/ccodec_gifmodule.cpp b/core/fxcodec/codec/ccodec_gifmodule.cpp index 9911d0f5e0..7bf216e393 100644 --- a/core/fxcodec/codec/ccodec_gifmodule.cpp +++ b/core/fxcodec/codec/ccodec_gifmodule.cpp @@ -57,12 +57,12 @@ CFX_GifDecodeStatus CCodec_GifModule::LoadFrame(Context* pContext, CFX_DIBAttribute* pAttribute) { auto* context = static_cast<CFX_GifContext*>(pContext); CFX_GifDecodeStatus ret = context->LoadFrame(frame_num); - if (ret != CFX_GifDecodeStatus::Success || !pAttribute) + if (ret != CFX_GifDecodeStatus::Success) return ret; - pAttribute->m_nGifLeft = context->images_[frame_num]->image_info.left; - pAttribute->m_nGifTop = context->images_[frame_num]->image_info.top; - pAttribute->m_fAspectRatio = context->pixel_aspect_; + if (pAttribute) + pAttribute->m_fAspectRatio = context->pixel_aspect_; + return CFX_GifDecodeStatus::Success; } |