examples/encode_video: use the AVFrame API for allocating the frame
authorAnton Khirnov <anton@khirnov.net>
Thu, 20 Oct 2016 09:03:20 +0000 (11:03 +0200)
committerAnton Khirnov <anton@khirnov.net>
Wed, 2 Nov 2016 09:20:01 +0000 (10:20 +0100)
It is more efficient and so preferred over allocating the buffers
manually.

doc/examples/encode_video.c

index b955cce..3fd2d56 100644 (file)
@@ -88,16 +88,16 @@ int main(int argc, char **argv)
         exit(1);
     }
 
-    ret = av_image_alloc(picture->data, picture->linesize, c->width, c->height,
-                         c->pix_fmt, 32);
-    if (ret < 0) {
-        fprintf(stderr, "could not alloc raw picture buffer\n");
-        exit(1);
-    }
     picture->format = c->pix_fmt;
     picture->width  = c->width;
     picture->height = c->height;
 
+    ret = av_frame_get_buffer(picture, 32);
+    if (ret < 0) {
+        fprintf(stderr, "could not alloc the frame data\n");
+        exit(1);
+    }
+
     /* encode 1 second of video */
     for(i=0;i<25;i++) {
         av_init_packet(&pkt);
@@ -105,6 +105,12 @@ int main(int argc, char **argv)
         pkt.size = 0;
 
         fflush(stdout);
+
+        /* make sure the frame data is writable */
+        ret = av_frame_make_writable(picture);
+        if (ret < 0)
+            exit(1);
+
         /* prepare a dummy image */
         /* Y */
         for(y=0;y<c->height;y++) {
@@ -159,7 +165,6 @@ int main(int argc, char **argv)
     fclose(f);
 
     avcodec_free_context(&c);
-    av_freep(&picture->data[0]);
     av_frame_free(&picture);
 
     return 0;