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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: core/fxcodec/codec/fx_codec_jpeg.cpp

Issue 2454263002: Fix more FX_BOOL / int noise in core/fpdfsdk (Closed)
Patch Set: Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « core/fxcodec/codec/ccodec_jpegmodule.h ('k') | core/fxge/win32/fx_win32_device.cpp » ('j') | fpdfsdk/javascript/Document.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: core/fxcodec/codec/fx_codec_jpeg.cpp
diff --git a/core/fxcodec/codec/fx_codec_jpeg.cpp b/core/fxcodec/codec/fx_codec_jpeg.cpp
index 7670516a250fa4e8905f860767a0078f87efe9c1..3d246566b2ab2171cb2e47b466440341c166ecfe 100644
--- a/core/fxcodec/codec/fx_codec_jpeg.cpp
+++ b/core/fxcodec/codec/fx_codec_jpeg.cpp
@@ -449,12 +449,12 @@ int CCodec_JpegModule::ReadHeader(FXJPEG_Context* ctx,
return 0;
}
-int CCodec_JpegModule::StartScanline(FXJPEG_Context* ctx, int down_scale) {
+FX_BOOL CCodec_JpegModule::StartScanline(FXJPEG_Context* ctx, int down_scale) {
if (setjmp(ctx->m_JumpMark) == -1)
return 0;
Lei Zhang 2016/10/28 00:50:20 return false?
ctx->m_Info.scale_denom = down_scale;
- return jpeg_start_decompress(&ctx->m_Info);
+ return !!jpeg_start_decompress(&ctx->m_Info);
}
FX_BOOL CCodec_JpegModule::ReadScanline(FXJPEG_Context* ctx,
« no previous file with comments | « core/fxcodec/codec/ccodec_jpegmodule.h ('k') | core/fxge/win32/fx_win32_device.cpp » ('j') | fpdfsdk/javascript/Document.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698