[FFmpeg-cvslog] zerocodec: Cosmetics

Derek Buitenhuis git at videolan.org
Sat Aug 4 21:25:25 CEST 2012


ffmpeg | branch: master | Derek Buitenhuis <derek.buitenhuis at gmail.com> | Fri Aug  3 21:41:24 2012 -0400| [d6124d0d142a9246b3580a3eccb671d727b7e504] | committer: Derek Buitenhuis

zerocodec: Cosmetics

Be consistent with error messages and code formatting.

Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=d6124d0d142a9246b3580a3eccb671d727b7e504
---

 libavcodec/zerocodec.c |   27 +++++++++++++--------------
 1 file changed, 13 insertions(+), 14 deletions(-)

diff --git a/libavcodec/zerocodec.c b/libavcodec/zerocodec.c
index 6581706..8d46bfd 100644
--- a/libavcodec/zerocodec.c
+++ b/libavcodec/zerocodec.c
@@ -23,7 +23,7 @@
 typedef struct {
     AVFrame  previous_frame;
     z_stream zstream;
-    int size;
+    int      size;
 } ZeroCodecContext;
 
 static int zerocodec_decode_frame(AVCodecContext *avctx, void *data,
@@ -33,7 +33,8 @@ static int zerocodec_decode_frame(AVCodecContext *avctx, void *data,
     AVFrame *pic         = avctx->coded_frame;
     AVFrame *prev_pic    = &zc->previous_frame;
     z_stream *zstream    = &zc->zstream;
-    uint8_t *prev = prev_pic->data[0], *dst;
+    uint8_t *prev        = prev_pic->data[0];
+    uint8_t *dst;
     int i, j, zret;
 
     pic->reference = 3;
@@ -43,7 +44,7 @@ static int zerocodec_decode_frame(AVCodecContext *avctx, void *data,
         pic->pict_type = AV_PICTURE_TYPE_I;
     } else {
         if (!prev) {
-            av_log(avctx, AV_LOG_ERROR, "Missing reference frame!\n");
+            av_log(avctx, AV_LOG_ERROR, "Missing reference frame.\n");
             return AVERROR_INVALIDDATA;
         }
         pic->key_frame = 0;
@@ -56,16 +57,15 @@ static int zerocodec_decode_frame(AVCodecContext *avctx, void *data,
     }
 
     zret = inflateReset(zstream);
-
     if (zret != Z_OK) {
-        av_log(avctx, AV_LOG_ERROR, "Could not reset inflate: %d\n", zret);
-        return AVERROR(EINVAL);
+        av_log(avctx, AV_LOG_ERROR, "Could not reset inflate: %d.\n", zret);
+        return AVERROR_INVALIDDATA;
     }
 
-    zstream->next_in   = avpkt->data;
-    zstream->avail_in  = avpkt->size;
+    zstream->next_in  = avpkt->data;
+    zstream->avail_in = avpkt->size;
 
-    dst  = pic->data[0];
+    dst = pic->data[0];
 
     /**
      * ZeroCodec has very simple interframe compression. If a value
@@ -75,11 +75,12 @@ static int zerocodec_decode_frame(AVCodecContext *avctx, void *data,
     for (i = 0; i < avctx->height; i++) {
         zstream->next_out  = dst;
         zstream->avail_out = avctx->width << 1;
+
         zret = inflate(zstream, Z_SYNC_FLUSH);
         if (zret != Z_OK && zret != Z_STREAM_END) {
             av_log(avctx, AV_LOG_ERROR,
-                   "Inflate failed with return code: %d\n", zret);
-            return AVERROR(EINVAL);
+                   "Inflate failed with return code: %d.\n", zret);
+            return AVERROR_INVALIDDATA;
         }
 
         if (!(avpkt->flags & AV_PKT_FLAG_KEY))
@@ -137,14 +138,12 @@ static av_cold int zerocodec_decode_init(AVCodecContext *avctx)
     zstream->opaque = Z_NULL;
 
     zret = inflateInit(zstream);
-
     if (zret != Z_OK) {
-        av_log(avctx, AV_LOG_ERROR, "Could not initialize inflate: %d\n", zret);
+        av_log(avctx, AV_LOG_ERROR, "Could not initialize inflate: %d.\n", zret);
         return AVERROR(ENOMEM);
     }
 
     avctx->coded_frame = avcodec_alloc_frame();
-
     if (!avctx->coded_frame) {
         av_log(avctx, AV_LOG_ERROR, "Could not allocate frame buffer.\n");
         zerocodec_decode_close(avctx);



More information about the ffmpeg-cvslog mailing list