img2enc: Refactor the atomic renaming code

[ffmpeg] / libavformat / msnwc_tcp.c

diff --git a/libavformat/msnwc_tcp.c b/libavformat/msnwc_tcp.c
index 0c12ea60ef4a702ebce1ba63ac9c02037e842970..6b9589d916809e0cf2e779bb05c455f287d82995 100644 (file)
--- a/libavformat/msnwc_tcp.c
+++ b/libavformat/msnwc_tcp.c
@@ -20,6 +20,7 @@
 
 #include "libavcodec/bytestream.h"
 #include "avformat.h"
+#include "internal.h"
 
 #define HEADER_SIZE         24
 
@@ -39,25 +40,26 @@ static int msnwc_tcp_probe(AVProbeData *p)
 {
     int i;
 
-    for(i = 0 ; i + HEADER_SIZE <= p->buf_size ; i++) {
+    for (i = 0; i + HEADER_SIZE <= p->buf_size; i++) {
         uint16_t width, height;
         uint32_t fourcc;
-        const uint8_t *bytestream = p->buf+i;
+        const uint8_t *bytestream = p->buf + i;
 
-        if(bytestream_get_le16(&bytestream) != HEADER_SIZE)
+        if (bytestream_get_le16(&bytestream) != HEADER_SIZE)
             continue;
         width  = bytestream_get_le16(&bytestream);
         height = bytestream_get_le16(&bytestream);
-        if(!(width==320 && height==240) && !(width==160 && height==120))
+        if (!(width == 320 &&
+              height == 240) && !(width == 160 && height == 120))
             continue;
         bytestream += 2; // keyframe
         bytestream += 4; // size
-        fourcc = bytestream_get_le32(&bytestream);
-        if(fourcc != MKTAG('M', 'L', '2', '0'))
+        fourcc      = bytestream_get_le32(&bytestream);
+        if (fourcc != MKTAG('M', 'L', '2', '0'))
             continue;
 
-        if(i) {
-            if(i < 14)  /* starts with SwitchBoard connection info */
+        if (i) {
+            if (i < 14) /* starts with SwitchBoard connection info */
                 return AVPROBE_SCORE_MAX / 2;
             else        /* starts in the middle of stream */
                 return AVPROBE_SCORE_MAX / 3;
@@ -66,33 +68,33 @@ static int msnwc_tcp_probe(AVProbeData *p)
         }
     }
 
-    return -1;
+    return 0;
 }
 
-static int msnwc_tcp_read_header(AVFormatContext *ctx, AVFormatParameters *ap)
+static int msnwc_tcp_read_header(AVFormatContext *ctx)
 {
     AVIOContext *pb = ctx->pb;
-    AVCodecContext *codec;
+    AVCodecParameters *par;
     AVStream *st;
 
     st = avformat_new_stream(ctx, NULL);
-    if(!st)
+    if (!st)
         return AVERROR(ENOMEM);
 
-    codec = st->codec;
-    codec->codec_type = AVMEDIA_TYPE_VIDEO;
-    codec->codec_id = CODEC_ID_MIMIC;
-    codec->codec_tag = MKTAG('M', 'L', '2', '0');
+    par             = st->codecpar;
+    par->codec_type = AVMEDIA_TYPE_VIDEO;
+    par->codec_id   = AV_CODEC_ID_MIMIC;
+    par->codec_tag  = MKTAG('M', 'L', '2', '0');
 
-    av_set_pts_info(st, 32, 1, 1000);
+    avpriv_set_pts_info(st, 32, 1, 1000);
 
     /* Some files start with "connected\r\n\r\n".
      * So skip until we find the first byte of struct size */
-    while(avio_r8(pb) != HEADER_SIZE && !pb->eof_reached);
+    while (avio_r8(pb) != HEADER_SIZE && !pb->eof_reached) ;
 
-    if(pb->eof_reached) {
+    if (pb->eof_reached) {
         av_log(ctx, AV_LOG_ERROR, "Could not find valid start.");
-        return -1;
+        return AVERROR_INVALIDDATA;
     }
 
     return 0;
@@ -103,37 +105,41 @@ static int msnwc_tcp_read_packet(AVFormatContext *ctx, AVPacket *pkt)
     AVIOContext *pb = ctx->pb;
     uint16_t keyframe;
     uint32_t size, timestamp;
+    int ret;
 
     avio_skip(pb, 1); /* one byte has been read ahead */
     avio_skip(pb, 2);
     avio_skip(pb, 2);
     keyframe = avio_rl16(pb);
-    size = avio_rl32(pb);
+    size     = avio_rl32(pb);
     avio_skip(pb, 4);
     avio_skip(pb, 4);
     timestamp = avio_rl32(pb);
 
-    if(!size || av_get_packet(pb, pkt, size) != size)
-        return -1;
+    if (!size)
+        return AVERROR_INVALIDDATA;
+
+    if ((ret = av_get_packet(pb, pkt, size)) < 0)
+        return ret;
 
     avio_skip(pb, 1); /* Read ahead one byte of struct size like read_header */
 
-    pkt->pts = timestamp;
-    pkt->dts = timestamp;
+    pkt->pts          = timestamp;
+    pkt->dts          = timestamp;
     pkt->stream_index = 0;
 
     /* Some aMsn generated videos (or was it Mercury Messenger?) don't set
      * this bit and rely on the codec to get keyframe information */
-    if(keyframe&1)
+    if (keyframe & 1)
         pkt->flags |= AV_PKT_FLAG_KEY;
 
     return HEADER_SIZE + size;
 }
 
 AVInputFormat ff_msnwc_tcp_demuxer = {
-    .name           = "msnwctcp",
-    .long_name      = NULL_IF_CONFIG_SMALL("MSN TCP Webcam stream"),
-    .read_probe     = msnwc_tcp_probe,
-    .read_header    = msnwc_tcp_read_header,
-    .read_packet    = msnwc_tcp_read_packet,
+    .name        = "msnwctcp",
+    .long_name   = NULL_IF_CONFIG_SMALL("MSN TCP Webcam stream"),
+    .read_probe  = msnwc_tcp_probe,
+    .read_header = msnwc_tcp_read_header,
+    .read_packet = msnwc_tcp_read_packet,
 };