]> git.sesse.net Git - ffmpeg/blobdiff - libavcodec/sgienc.c
aacenc: Write correct length for long identification strings.
[ffmpeg] / libavcodec / sgienc.c
index 09d3c06c7a3d652e1b1081717b4f771d651d25fa..9d02eea136d623b576a00deaa876c10fb0dfaf0b 100644 (file)
@@ -2,20 +2,20 @@
  * SGI image encoder
  * Todd Kirby <doubleshot@pacbell.net>
  *
- * This file is part of FFmpeg.
+ * This file is part of Libav.
  *
- * FFmpeg is free software; you can redistribute it and/or
+ * Libav is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
  * License as published by the Free Software Foundation; either
  * version 2.1 of the License, or (at your option) any later version.
  *
- * FFmpeg is distributed in the hope that it will be useful,
+ * Libav is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Lesser General Public License for more details.
  *
  * You should have received a copy of the GNU Lesser General Public
- * License along with FFmpeg; if not, write to the Free Software
+ * License along with Libav; if not, write to the Free Software
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
@@ -31,7 +31,8 @@ typedef struct SgiContext {
     AVFrame picture;
 } SgiContext;
 
-static av_cold int encode_init(AVCodecContext *avctx){
+static av_cold int encode_init(AVCodecContext *avctx)
+{
     SgiContext *s = avctx->priv_data;
 
     avcodec_get_frame_defaults(&s->picture);
@@ -41,7 +42,8 @@ static av_cold int encode_init(AVCodecContext *avctx){
 }
 
 static int encode_frame(AVCodecContext *avctx, unsigned char *buf,
-                        int buf_size, void *data) {
+                        int buf_size, void *data)
+{
     SgiContext *s = avctx->priv_data;
     AVFrame * const p = &s->picture;
     uint8_t *offsettab, *lengthtab, *in_buf, *encode_buf;
@@ -50,27 +52,27 @@ static int encode_frame(AVCodecContext *avctx, unsigned char *buf,
     unsigned char *orig_buf = buf, *end_buf = buf + buf_size;
 
     *p = *(AVFrame*)data;
-    p->pict_type = FF_I_TYPE;
+    p->pict_type = AV_PICTURE_TYPE_I;
     p->key_frame = 1;
 
-    width = avctx->width;
+    width  = avctx->width;
     height = avctx->height;
 
     switch (avctx->pix_fmt) {
-        case PIX_FMT_GRAY8:
-            dimension = SGI_SINGLE_CHAN;
-            depth = SGI_GRAYSCALE;
-            break;
-        case PIX_FMT_RGB24:
-            dimension = SGI_MULTI_CHAN;
-            depth = SGI_RGB;
-            break;
-        case PIX_FMT_RGBA:
-            dimension = SGI_MULTI_CHAN;
-            depth = SGI_RGBA;
-            break;
-        default:
-            return AVERROR_INVALIDDATA;
+    case PIX_FMT_GRAY8:
+        dimension = SGI_SINGLE_CHAN;
+        depth     = SGI_GRAYSCALE;
+        break;
+    case PIX_FMT_RGB24:
+        dimension = SGI_MULTI_CHAN;
+        depth     = SGI_RGB;
+        break;
+    case PIX_FMT_RGBA:
+        dimension = SGI_MULTI_CHAN;
+        depth     = SGI_RGBA;
+        break;
+    default:
+        return AVERROR_INVALIDDATA;
     }
 
     tablesize = depth * height * 4;
@@ -83,7 +85,7 @@ static int encode_frame(AVCodecContext *avctx, unsigned char *buf,
 
     /* Encode header. */
     bytestream_put_be16(&buf, SGI_MAGIC);
-    bytestream_put_byte(&buf, 1); /* RLE */
+    bytestream_put_byte(&buf, avctx->coder_type != FF_CODER_TYPE_RAW); /* RLE 1 - VERBATIM 0*/
     bytestream_put_byte(&buf, 1); /* bytes_per_channel */
     bytestream_put_be16(&buf, dimension);
     bytestream_put_be16(&buf, width);
@@ -106,51 +108,64 @@ static int encode_frame(AVCodecContext *avctx, unsigned char *buf,
     buf += 404;
     offsettab = buf;
 
-    /* Skip RLE offset table. */
-    buf += tablesize;
-    lengthtab = buf;
+    if (avctx->coder_type  != FF_CODER_TYPE_RAW) {
+        /* Skip RLE offset table. */
+        buf += tablesize;
+        lengthtab = buf;
 
-    /* Skip RLE length table. */
-    buf += tablesize;
+        /* Skip RLE length table. */
+        buf += tablesize;
 
-    /* Make an intermediate consecutive buffer. */
-    if ((encode_buf = av_malloc(width)) == NULL)
-        return -1;
+        /* Make an intermediate consecutive buffer. */
+        if (!(encode_buf = av_malloc(width)))
+            return -1;
+
+        for (z = 0; z < depth; z++) {
+            in_buf = p->data[0] + p->linesize[0] * (height - 1) + z;
 
-    for (z = 0; z < depth; z++) {
-        in_buf = p->data[0] + p->linesize[0] * (height - 1) + z;
+            for (y = 0; y < height; y++) {
+                bytestream_put_be32(&offsettab, buf - orig_buf);
 
-        for (y = 0; y < height; y++) {
-            bytestream_put_be32(&offsettab, buf - orig_buf);
+                for (x = 0; x < width; x++)
+                    encode_buf[x] = in_buf[depth * x];
 
-            for (x = 0; x < width; x++)
-                encode_buf[x] = in_buf[depth * x];
+                if ((length = ff_rle_encode(buf, end_buf - buf - 1, encode_buf, 1, width, 0, 0, 0x80, 0)) < 1) {
+                    av_free(encode_buf);
+                    return -1;
+                }
 
-            if((length = ff_rle_encode(buf, end_buf - buf - 1, encode_buf, 1, width, 0, 0, 0x80, 0)) < 1) {
-                av_free(encode_buf);
-                return -1;
+                buf += length;
+                bytestream_put_byte(&buf, 0);
+                bytestream_put_be32(&lengthtab, length + 1);
+                in_buf -= p->linesize[0];
             }
+        }
+
+        av_free(encode_buf);
+    } else {
+        for (z = 0; z < depth; z++) {
+            in_buf = p->data[0] + p->linesize[0] * (height - 1) + z;
 
-            buf += length;
-            bytestream_put_byte(&buf, 0);
-            bytestream_put_be32(&lengthtab, length + 1);
-            in_buf -= p->linesize[0];
+            for (y = 0; y < height; y++) {
+                for (x = 0; x < width * depth; x += depth)
+                    bytestream_put_byte(&buf, in_buf[x]);
+
+                in_buf -= p->linesize[0];
+            }
         }
     }
 
-    av_free(encode_buf);
     /* total length */
     return buf - orig_buf;
 }
 
-AVCodec sgi_encoder = {
-    "sgi",
-    CODEC_TYPE_VIDEO,
-    CODEC_ID_SGI,
-    sizeof(SgiContext),
-    encode_init,
-    encode_frame,
-    NULL,
-    .pix_fmts= (enum PixelFormat[]){PIX_FMT_RGB24, PIX_FMT_RGBA, PIX_FMT_PAL8, PIX_FMT_GRAY8, -1},
+AVCodec ff_sgi_encoder = {
+    .name           = "sgi",
+    .type           = AVMEDIA_TYPE_VIDEO,
+    .id             = CODEC_ID_SGI,
+    .priv_data_size = sizeof(SgiContext),
+    .init           = encode_init,
+    .encode         = encode_frame,
+    .pix_fmts= (const enum PixelFormat[]){PIX_FMT_RGB24, PIX_FMT_RGBA, PIX_FMT_GRAY8, PIX_FMT_NONE},
+    .long_name= NULL_IF_CONFIG_SMALL("SGI image"),
 };
-