[FFmpeg-cvslog] libavcodec/exr : indent gray support patch
Martin Vignali
git at videolan.org
Thu Jul 7 00:12:27 CEST 2016
ffmpeg | branch: master | Martin Vignali <martin.vignali at gmail.com> | Mon Jun 27 23:57:06 2016 +0200| [50747d6b1f25a2d1feafb5e413c7e22bad4c450c] | committer: Michael Niedermayer
libavcodec/exr : indent gray support patch
Reviewed-by: Paul B Mahol <onemda at gmail.com>
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=50747d6b1f25a2d1feafb5e413c7e22bad4c450c
---
libavcodec/exr.c | 39 ++++++++++++++++++++-------------------
1 file changed, 20 insertions(+), 19 deletions(-)
diff --git a/libavcodec/exr.c b/libavcodec/exr.c
index 3c05873..e226fd7 100644
--- a/libavcodec/exr.c
+++ b/libavcodec/exr.c
@@ -1146,10 +1146,10 @@ static int decode_block(AVCodecContext *avctx, void *tdata,
}
if (!s->is_luma) {
- channel_buffer[0] = src + td->xsize * s->channel_offsets[0];
- channel_buffer[1] = src + td->xsize * s->channel_offsets[1];
- channel_buffer[2] = src + td->xsize * s->channel_offsets[2];
- rgb_channel_count = 3;
+ channel_buffer[0] = src + td->xsize * s->channel_offsets[0];
+ channel_buffer[1] = src + td->xsize * s->channel_offsets[1];
+ channel_buffer[2] = src + td->xsize * s->channel_offsets[2];
+ rgb_channel_count = 3;
} else { /* put y data in the first channel_buffer */
channel_buffer[0] = src + td->xsize * s->channel_offsets[1];
rgb_channel_count = 1;
@@ -1380,11 +1380,12 @@ static int decode_header(EXRContext *s)
!strcmp(ch_gb.buffer, "W")){
channel_index = 2;
s->is_luma = 0;
- } else if (!strcmp(ch_gb.buffer, "A"))
+ } else if (!strcmp(ch_gb.buffer, "A")) {
channel_index = 3;
- else
+ } else {
av_log(s->avctx, AV_LOG_WARNING,
"Unsupported channel %.256s.\n", ch_gb.buffer);
+ }
}
/* skip until you get a 0 */
@@ -1443,17 +1444,17 @@ static int decode_header(EXRContext *s)
/* Check if all channels are set with an offset or if the channels
* are causing an overflow */
if (!s->is_luma){/* if we expected to have at least 3 channels */
- if (FFMIN3(s->channel_offsets[0],
- s->channel_offsets[1],
- s->channel_offsets[2]) < 0) {
- if (s->channel_offsets[0] < 0)
- av_log(s->avctx, AV_LOG_ERROR, "Missing red channel.\n");
- if (s->channel_offsets[1] < 0)
- av_log(s->avctx, AV_LOG_ERROR, "Missing green channel.\n");
- if (s->channel_offsets[2] < 0)
- av_log(s->avctx, AV_LOG_ERROR, "Missing blue channel.\n");
- return AVERROR_INVALIDDATA;
- }
+ if (FFMIN3(s->channel_offsets[0],
+ s->channel_offsets[1],
+ s->channel_offsets[2]) < 0) {
+ if (s->channel_offsets[0] < 0)
+ av_log(s->avctx, AV_LOG_ERROR, "Missing red channel.\n");
+ if (s->channel_offsets[1] < 0)
+ av_log(s->avctx, AV_LOG_ERROR, "Missing green channel.\n");
+ if (s->channel_offsets[2] < 0)
+ av_log(s->avctx, AV_LOG_ERROR, "Missing blue channel.\n");
+ return AVERROR_INVALIDDATA;
+ }
}
// skip one last byte and update main gb
@@ -1606,13 +1607,13 @@ static int decode_frame(AVCodecContext *avctx, void *data,
case EXR_HALF:
if (s->channel_offsets[3] >= 0) {
if (!s->is_luma) {
- avctx->pix_fmt = AV_PIX_FMT_RGBA64;
+ avctx->pix_fmt = AV_PIX_FMT_RGBA64;
} else {
avctx->pix_fmt = AV_PIX_FMT_YA16;
}
} else {
if (!s->is_luma) {
- avctx->pix_fmt = AV_PIX_FMT_RGB48;
+ avctx->pix_fmt = AV_PIX_FMT_RGB48;
} else {
avctx->pix_fmt = AV_PIX_FMT_GRAY16;
}
More information about the ffmpeg-cvslog
mailing list