Merge remote-tracking branch 'qatar/master'
authorMichael Niedermayer <michaelni@gmx.at>
Fri, 27 May 2011 21:47:05 +0000 (23:47 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Fri, 27 May 2011 21:48:22 +0000 (23:48 +0200)
* qatar/master: (29 commits)
  ARM: disable ff_vector_fmul_vfp on VFPv3 systems
  ARM: check for VFPv3
  swscale: Remove unused variables in x86 code.
  doc: Drop DJGPP section, Libav now compiles out-of-the-box on FreeDOS.
  x86: Add appropriate ifdefs around certain AVX functions.
  cmdutils: use sws_freeContext() instead of av_freep().
  swscale: delay allocation of formatConvBuffer().
  swscale: fix build with --disable-swscale-alpha.
  movenc: Deprecate the global RTP hinting flag, use a private AVOption instead
  movenc: Add an AVClass for setting muxer specific options
  swscale: fix non-bitexact yuv2yuv[X2]() MMX/MMX2 functions.
  configure: report yasm/nasm presence properly
  tcp: make connect() timeout properly
  rawdec: factor video demuxer definitions into a macro.
  rtspdec: add initial_pause private option.
  lavf: deprecate AVFormatParameters.width/height.
  tty: add video_size private option.
  rawdec: add video_size private option.
  x11grab: add video_size private option.
  x11grab: factorize returning error codes.
  ...

Merged-by: Michael Niedermayer <michaelni@gmx.at>
32 files changed:
1  2 
cmdutils.c
configure
doc/general.texi
libavcodec/arm/dsputil_init_vfp.c
libavcodec/x86/fft_mmx.asm
libavcodec/x86/fft_sse.c
libavdevice/bktr.c
libavdevice/libdc1394.c
libavdevice/v4l2.c
libavdevice/vfwcap.c
libavdevice/x11grab.c
libavformat/avformat.h
libavformat/cavsvideodec.c
libavformat/diracdec.c
libavformat/dnxhddec.c
libavformat/h261dec.c
libavformat/h263dec.c
libavformat/h264dec.c
libavformat/id3v2.c
libavformat/m4vdec.c
libavformat/movenc.c
libavformat/movenc.h
libavformat/mpegvideodec.c
libavformat/options.c
libavformat/rawdec.c
libavformat/rawdec.h
libavformat/rawvideodec.c
libavformat/rtsp.h
libavformat/rtspdec.c
libavformat/tcp.c
libavformat/tty.c
libavformat/version.h

diff --cc cmdutils.c
@@@ -76,11 -76,16 +76,12 @@@ void uninit_opts(void
      av_freep(&avformat_opts->key);
      av_freep(&avformat_opts);
  #if CONFIG_SWSCALE
-     av_freep(&sws_opts);
+     sws_freeContext(sws_opts);
+     sws_opts = NULL;
  #endif
      for (i = 0; i < opt_name_count; i++) {
 -        //opt_values are only stored for codec-specific options in which case
 -        //both the name and value are dup'd
 -        if (opt_values[i]) {
 -            av_freep(&opt_names[i]);
 -            av_freep(&opt_values[i]);
 -        }
 +        av_freep(&opt_names[i]);
 +        av_freep(&opt_values[i]);
      }
      av_freep(&opt_names);
      av_freep(&opt_values);
diff --cc configure
Simple merge
@@@ -762,15 -754,9 +762,9 @@@ performance on systems without hardwar
  
  Using a cross-compiler is preferred for various reasons.
  
- @subsection DJGPP
- FFmpeg cannot be compiled because of broken system headers, add
- @code{--extra-cflags=-U__STRICT_ANSI__} to the configure options as a
- workaround.
  @section OS/2
  
 -For information about compiling Libav on OS/2 see
 +For information about compiling FFmpeg on OS/2 see
  @url{http://www.edm2.com/index.php/FFmpeg}.
  
  @section Unix-like
Simple merge
Simple merge
Simple merge
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
 -#include "libavformat/avformat.h"
  #include "libavutil/log.h"
  #include "libavutil/opt.h"
+ #include "libavutil/parseutils.h"
  #if HAVE_DEV_BKTR_IOCTL_METEOR_H && HAVE_DEV_BKTR_IOCTL_BT848_H
  # include <dev/bktr/ioctl_meteor.h>
  # include <dev/bktr/ioctl_bt848.h>
   */
  
  #include "config.h"
 -#include "libavformat/avformat.h"
  #include "libavutil/log.h"
  #include "libavutil/opt.h"
 +#include "avdevice.h"
 +
 +#include <stdlib.h>
 +#include <string.h>
+ #include "libavutil/parseutils.h"
  
 -#if HAVE_LIBDC1394_2
  #include <dc1394/dc1394.h>
 -#elif HAVE_LIBDC1394_1
 -#include <libraw1394/raw1394.h>
 -#include <libdc1394/dc1394_control.h>
 -
 -#define DC1394_VIDEO_MODE_320x240_YUV422 MODE_320x240_YUV422
 -#define DC1394_VIDEO_MODE_640x480_YUV411 MODE_640x480_YUV411
 -#define DC1394_VIDEO_MODE_640x480_YUV422 MODE_640x480_YUV422
 -#define DC1394_FRAMERATE_1_875 FRAMERATE_1_875
 -#define DC1394_FRAMERATE_3_75  FRAMERATE_3_75
 -#define DC1394_FRAMERATE_7_5   FRAMERATE_7_5
 -#define DC1394_FRAMERATE_15    FRAMERATE_15
 -#define DC1394_FRAMERATE_30    FRAMERATE_30
 -#define DC1394_FRAMERATE_60    FRAMERATE_60
 -#define DC1394_FRAMERATE_120   FRAMERATE_120
 -#define DC1394_FRAMERATE_240   FRAMERATE_240
 -#endif
  
  #undef free
  
@@@ -38,8 -52,15 +39,9 @@@ typedef struct dc1394_data 
      dc1394_t *d;
      dc1394camera_t *camera;
      dc1394video_frame_t *frame;
 -#endif
      int current_frame;
      int fps;
+     char *video_size;       /**< String describing video size, set by a private option. */
  
      AVPacket packet;
  } dc1394_data;
@@@ -76,7 -92,13 +78,10 @@@ struct dc1394_frame_rate 
      { 0, 0 } /* gotta be the last one */
  };
  
+ #define OFFSET(x) offsetof(dc1394_data, x)
+ #define DEC AV_OPT_FLAG_DECODING_PARAM
  static const AVOption options[] = {
 -#if HAVE_LIBDC1394_1
 -    { "channel", "", offsetof(dc1394_data, channel), FF_OPT_TYPE_INT, {.dbl = 0}, 0, INT_MAX, AV_OPT_FLAG_DECODING_PARAM },
 -#endif
+     { "video_size", "A string describing frame size, such as 640x480 or hd720.", OFFSET(video_size), FF_OPT_TYPE_STRING, {.str = "qvga"}, 0, 0, DEC },
      { NULL },
  };
  
@@@ -87,22 -109,187 +92,23 @@@ static const AVClass libdc1394_class = 
      .version    = LIBAVUTIL_VERSION_INT,
  };
  
 -
 -static inline int dc1394_read_common(AVFormatContext *c, AVFormatParameters *ap,
 -                                     struct dc1394_frame_format **select_fmt, struct dc1394_frame_rate **select_fps)
 -{
 -    dc1394_data* dc1394 = c->priv_data;
 -    AVStream* vst;
 -    struct dc1394_frame_format *fmt;
 -    struct dc1394_frame_rate *fps;
 -    enum PixelFormat pix_fmt = ap->pix_fmt == PIX_FMT_NONE ? PIX_FMT_UYVY422 : ap->pix_fmt; /* defaults */
 -    int width, height;
 -    int frame_rate           = !ap->time_base.num ? 30000 : av_rescale(1000, ap->time_base.den, ap->time_base.num);
 -    int ret = 0;
 -
 -    if ((ret = av_parse_video_size(&width, &height, dc1394->video_size)) < 0) {
 -        av_log(c, AV_LOG_ERROR, "Couldn't parse video size.\n");
 -        goto out;
 -    }
 -#if FF_API_FORMAT_PARAMETERS
 -    if (ap->width > 0)
 -        width = ap->width;
 -    if (ap->height > 0)
 -        height = ap->height;
 -#endif
 -
 -    for (fmt = dc1394_frame_formats; fmt->width; fmt++)
 -         if (fmt->pix_fmt == pix_fmt && fmt->width == width && fmt->height == height)
 -             break;
 -
 -    for (fps = dc1394_frame_rates; fps->frame_rate; fps++)
 -         if (fps->frame_rate == frame_rate)
 -             break;
 -
 -    if (!fps->frame_rate || !fmt->width) {
 -        av_log(c, AV_LOG_ERROR, "Can't find matching camera format for %s, %dx%d@%d:1000fps\n", avcodec_get_pix_fmt_name(pix_fmt),
 -                                                                                                width, height, frame_rate);
 -        ret = AVERROR(EINVAL);
 -        goto out;
 -    }
 -
 -    /* create a video stream */
 -    vst = av_new_stream(c, 0);
 -    if (!vst) {
 -        ret = AVERROR(ENOMEM);
 -        goto out;
 -    }
 -    av_set_pts_info(vst, 64, 1, 1000);
 -    vst->codec->codec_type = AVMEDIA_TYPE_VIDEO;
 -    vst->codec->codec_id = CODEC_ID_RAWVIDEO;
 -    vst->codec->time_base.den = fps->frame_rate;
 -    vst->codec->time_base.num = 1000;
 -    vst->codec->width = fmt->width;
 -    vst->codec->height = fmt->height;
 -    vst->codec->pix_fmt = fmt->pix_fmt;
 -
 -    /* packet init */
 -    av_init_packet(&dc1394->packet);
 -    dc1394->packet.size = avpicture_get_size(fmt->pix_fmt, fmt->width, fmt->height);
 -    dc1394->packet.stream_index = vst->index;
 -    dc1394->packet.flags |= AV_PKT_FLAG_KEY;
 -
 -    dc1394->current_frame = 0;
 -    dc1394->fps = fps->frame_rate;
 -
 -    vst->codec->bit_rate = av_rescale(dc1394->packet.size * 8, fps->frame_rate, 1000);
 -    *select_fps = fps;
 -    *select_fmt = fmt;
 -out:
 -    av_freep(&dc1394->video_size);
 -    return ret;
 -}
 -
 -#if HAVE_LIBDC1394_1
 -static int dc1394_v1_read_header(AVFormatContext *c, AVFormatParameters * ap)
 -{
 -    dc1394_data* dc1394 = c->priv_data;
 -    AVStream* vst;
 -    nodeid_t* camera_nodes;
 -    int res;
 -    struct dc1394_frame_format *fmt = NULL;
 -    struct dc1394_frame_rate *fps = NULL;
 -
 -    if (dc1394_read_common(c,ap,&fmt,&fps) != 0)
 -        return -1;
 -
 -#if FF_API_FORMAT_PARAMETERS
 -    if (ap->channel)
 -        dc1394->channel = ap->channel;
 -#endif
 -
 -    /* Now let us prep the hardware. */
 -    dc1394->handle = dc1394_create_handle(0); /* FIXME: gotta have ap->port */
 -    if (!dc1394->handle) {
 -        av_log(c, AV_LOG_ERROR, "Can't acquire dc1394 handle on port %d\n", 0 /* ap->port */);
 -        goto out;
 -    }
 -    camera_nodes = dc1394_get_camera_nodes(dc1394->handle, &res, 1);
 -    if (!camera_nodes || camera_nodes[dc1394->channel] == DC1394_NO_CAMERA) {
 -        av_log(c, AV_LOG_ERROR, "There's no IIDC camera on the channel %d\n", dc1394->channel);
 -        goto out_handle;
 -    }
 -    res = dc1394_dma_setup_capture(dc1394->handle, camera_nodes[dc1394->channel],
 -                                   0,
 -                                   FORMAT_VGA_NONCOMPRESSED,
 -                                   fmt->frame_size_id,
 -                                   SPEED_400,
 -                                   fps->frame_rate_id, 8, 1,
 -                                   c->filename,
 -                                   &dc1394->camera);
 -    dc1394_free_camera_nodes(camera_nodes);
 -    if (res != DC1394_SUCCESS) {
 -        av_log(c, AV_LOG_ERROR, "Can't prepare camera for the DMA capture\n");
 -        goto out_handle;
 -    }
 -
 -    res = dc1394_start_iso_transmission(dc1394->handle, dc1394->camera.node);
 -    if (res != DC1394_SUCCESS) {
 -        av_log(c, AV_LOG_ERROR, "Can't start isochronous transmission\n");
 -        goto out_handle_dma;
 -    }
 -
 -    return 0;
 -
 -out_handle_dma:
 -    dc1394_dma_unlisten(dc1394->handle, &dc1394->camera);
 -    dc1394_dma_release_camera(dc1394->handle, &dc1394->camera);
 -out_handle:
 -    dc1394_destroy_handle(dc1394->handle);
 -out:
 -    return -1;
 -}
 -
 -static int dc1394_v1_read_packet(AVFormatContext *c, AVPacket *pkt)
 -{
 -    struct dc1394_data *dc1394 = c->priv_data;
 -    int res;
 -
 -    /* discard stale frame */
 -    if (dc1394->current_frame++) {
 -        if (dc1394_dma_done_with_buffer(&dc1394->camera) != DC1394_SUCCESS)
 -            av_log(c, AV_LOG_ERROR, "failed to release %d frame\n", dc1394->current_frame);
 -    }
 -
 -    res = dc1394_dma_single_capture(&dc1394->camera);
 -
 -    if (res == DC1394_SUCCESS) {
 -        dc1394->packet.data = (uint8_t *)(dc1394->camera.capture_buffer);
 -        dc1394->packet.pts = (dc1394->current_frame * 1000000) / dc1394->fps;
 -        res = dc1394->packet.size;
 -    } else {
 -        av_log(c, AV_LOG_ERROR, "DMA capture failed\n");
 -        dc1394->packet.data = NULL;
 -        res = -1;
 -    }
 -
 -    *pkt = dc1394->packet;
 -    return res;
 -}
 -
 -static int dc1394_v1_close(AVFormatContext * context)
 -{
 -    struct dc1394_data *dc1394 = context->priv_data;
 -
 -    dc1394_stop_iso_transmission(dc1394->handle, dc1394->camera.node);
 -    dc1394_dma_unlisten(dc1394->handle, &dc1394->camera);
 -    dc1394_dma_release_camera(dc1394->handle, &dc1394->camera);
 -    dc1394_destroy_handle(dc1394->handle);
 -
 -    return 0;
 -}
 -
 -#elif HAVE_LIBDC1394_2
 -static int dc1394_v2_read_header(AVFormatContext *c, AVFormatParameters * ap)
 +static int dc1394_read_header(AVFormatContext *c, AVFormatParameters * ap)
  {
      dc1394_data* dc1394 = c->priv_data;
 +    AVStream *vst;
 +    const struct dc1394_color_coding *cc;
 +    const struct dc1394_frame_rate *fr;
      dc1394camera_list_t *list;
 -    int res, i;
 -    struct dc1394_frame_format *fmt = NULL;
 -    struct dc1394_frame_rate *fps = NULL;
 -
 -    if (dc1394_read_common(c,ap,&fmt,&fps) != 0)
 -       return -1;
 +    dc1394video_modes_t video_modes;
 +    dc1394video_mode_t video_mode;
 +    dc1394framerates_t frame_rates;
 +    dc1394framerate_t frame_rate;
 +    uint32_t dc1394_width, dc1394_height, dc1394_color_coding;
 +    int rate, best_rate;
 +    int score, max_score;
 +    int final_width, final_height, final_pix_fmt, final_frame_rate;
 +    int res, i, j;
++    int ret=-1;
  
      /* Now let us prep the hardware. */
      dc1394->d = dc1394_new();
      /* Freeing list of cameras */
      dc1394_camera_free_list (list);
  
 +    /* Get the list of video modes supported by the camera. */
 +    res = dc1394_video_get_supported_modes (dc1394->camera, &video_modes);
 +    if (res != DC1394_SUCCESS) {
 +        av_log(c, AV_LOG_ERROR, "Could not get video formats.\n");
 +        goto out_camera;
 +    }
++
++    if (dc1394->video_size) {
++        if ((ret = av_parse_video_size(&ap->width, &ap->height, dc1394->video_size)) < 0) {
++            av_log(c, AV_LOG_ERROR, "Couldn't parse video size.\n");
++            goto out;
++        }
++    }
++    
 +    /* Choose the best mode. */
 +    rate = (ap->time_base.num ? av_rescale(1000, ap->time_base.den, ap->time_base.num) : -1);
 +    max_score = -1;
 +    for (i = 0; i < video_modes.num; i++) {
 +        if (video_modes.modes[i] == DC1394_VIDEO_MODE_EXIF
 +                || (video_modes.modes[i] >= DC1394_VIDEO_MODE_FORMAT7_MIN
 +                    && video_modes.modes[i] <= DC1394_VIDEO_MODE_FORMAT7_MAX)) {
 +            /* These modes are currently not supported as they would require
 +             * much more work. For the remaining modes, the functions
 +             * dc1394_get_image_size_from_video_mode and
 +             * dc1394_get_color_coding_from_video_mode do not need to query the
 +             * camera, and thus cannot fail. */
 +            continue;
 +        }
 +        dc1394_get_color_coding_from_video_mode (NULL, video_modes.modes[i],
 +                &dc1394_color_coding);
 +        for (cc = dc1394_color_codings; cc->pix_fmt != PIX_FMT_NONE; cc++)
 +            if (cc->coding == dc1394_color_coding)
 +                break;
 +        if (cc->pix_fmt == PIX_FMT_NONE) {
 +            /* We currently cannot handle this color coding. */
 +            continue;
 +        }
 +        /* Here we know that the mode is supported. Get its frame size and the list
 +         * of frame rates supported by the camera for this mode. This list is sorted
 +         * in ascending order according to libdc1394 example programs. */
 +        dc1394_get_image_size_from_video_mode (NULL, video_modes.modes[i],
 +                &dc1394_width, &dc1394_height);
 +        res = dc1394_video_get_supported_framerates (dc1394->camera, video_modes.modes[i],
 +                &frame_rates);
 +        if (res != DC1394_SUCCESS || frame_rates.num == 0) {
 +            av_log(c, AV_LOG_ERROR, "Cannot get frame rates for video mode.\n");
 +            goto out_camera;
 +        }
 +        /* Choose the best frame rate. */
 +        best_rate = -1;
 +        for (j = 0; j < frame_rates.num; j++) {
 +            for (fr = dc1394_frame_rates; fr->frame_rate; fr++) {
 +                if (fr->frame_rate_id == frame_rates.framerates[j]) {
 +                    break;
 +                }
 +            }
 +            if (!fr->frame_rate) {
 +                /* This frame rate is not supported. */
 +                continue;
 +            }
 +            best_rate = fr->frame_rate;
 +            frame_rate = fr->frame_rate_id;
 +            if (ap->time_base.num && rate == fr->frame_rate) {
 +                /* This is the requested frame rate. */
 +                break;
 +            }
 +        }
 +        if (best_rate == -1) {
 +            /* No supported rate found. */
 +            continue;
 +        }
 +        /* Here we know that both the mode and the rate are supported. Compute score. */
 +        if (ap->width && ap->height
 +                && (dc1394_width == ap->width && dc1394_height == ap->height)) {
 +            score = 110000;
 +        } else {
 +            score = dc1394_width * 10;  // 1600 - 16000
 +        }
 +        if (ap->pix_fmt == cc->pix_fmt) {
 +            score += 90000;
 +        } else {
 +            score += cc->score;         // 1000 - 1500
 +        }
 +        if (ap->time_base.num && rate == best_rate) {
 +            score += 70000;
 +        } else {
 +            score += best_rate / 1000;  // 1 - 240
 +        }
 +        if (score > max_score) {
 +            video_mode = video_modes.modes[i];
 +            final_width = dc1394_width;
 +            final_height = dc1394_height;
 +            final_pix_fmt = cc->pix_fmt;
 +            final_frame_rate = best_rate;
 +            max_score = score;
 +        }
 +    }
 +    if (max_score == -1) {
 +        av_log(c, AV_LOG_ERROR, "No suitable video mode / frame rate available.\n");
 +        goto out_camera;
 +    }
 +    if (ap->width && ap->height && !(ap->width == final_width && ap->height == final_height)) {
 +        av_log(c, AV_LOG_WARNING, "Requested frame size is not available, using fallback.\n");
 +    }
 +    if (ap->pix_fmt != PIX_FMT_NONE && ap->pix_fmt != final_pix_fmt) {
 +        av_log(c, AV_LOG_WARNING, "Requested pixel format is not supported, using fallback.\n");
 +    }
 +    if (ap->time_base.num && rate != final_frame_rate) {
 +        av_log(c, AV_LOG_WARNING, "Requested frame rate is not available, using fallback.\n");
 +    }
 +
 +    /* create a video stream */
 +    vst = av_new_stream(c, 0);
 +    if (!vst)
 +        goto out_camera;
 +    av_set_pts_info(vst, 64, 1, 1000);
 +    vst->codec->codec_type = AVMEDIA_TYPE_VIDEO;
 +    vst->codec->codec_id = CODEC_ID_RAWVIDEO;
 +    vst->codec->time_base.den = final_frame_rate;
 +    vst->codec->time_base.num = 1000;
 +    vst->codec->width = final_width;
 +    vst->codec->height = final_height;
 +    vst->codec->pix_fmt = final_pix_fmt;
 +
 +    /* packet init */
 +    av_init_packet(&dc1394->packet);
 +    dc1394->packet.size = avpicture_get_size(final_pix_fmt, final_width, final_height);
 +    dc1394->packet.stream_index = vst->index;
 +    dc1394->packet.flags |= AV_PKT_FLAG_KEY;
 +
 +    dc1394->current_frame = 0;
 +    dc1394->fps = final_frame_rate;
 +
 +    vst->codec->bit_rate = av_rescale(dc1394->packet.size * 8, final_frame_rate, 1000);
 +
      /* Select MAX Speed possible from the cam */
      if (dc1394->camera->bmode_capable>0) {
         dc1394_video_set_operation_mode(dc1394->camera, DC1394_OPERATION_MODE_1394B);
@@@ -290,10 -350,10 +304,10 @@@ out_camera
      dc1394_camera_free (dc1394->camera);
  out:
      dc1394_free(dc1394->d);
--    return -1;
++    return ret;
  }
  
 -static int dc1394_v2_read_packet(AVFormatContext *c, AVPacket *pkt)
 +static int dc1394_read_packet(AVFormatContext *c, AVPacket *pkt)
  {
      struct dc1394_data *dc1394 = c->priv_data;
      int res;
@@@ -45,7 -46,7 +45,8 @@@
  #include "libavutil/imgutils.h"
  #include "libavutil/log.h"
  #include "libavutil/opt.h"
 +#include "avdevice.h"
+ #include "libavutil/parseutils.h"
  
  static const int desired_video_buffers = 256;
  
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
 -#include "libavformat/avformat.h"
+ #include "libavutil/log.h"
+ #include "libavutil/opt.h"
+ #include "libavutil/parseutils.h"
  #include <windows.h>
  #include <vfw.h>
 +#include "avdevice.h"
  
  //#define DEBUG_VFW
  
   */
  
  #include "config.h"
 -#include "libavformat/avformat.h"
+ #include "libavutil/log.h"
+ #include "libavutil/opt.h"
+ #include "libavutil/parseutils.h"
  #include <time.h>
  #include <X11/X.h>
  #include <X11/Xlib.h>
@@@ -90,28 -95,41 +95,42 @@@ x11grab_read_header(AVFormatContext *s1
      int x_off = 0;
      int y_off = 0;
      int use_shm;
 -    char *param, *offset;
 +    char *dpyname, *offset;
+     int ret = 0;
  
 -    param = av_strdup(s1->filename);
 -    offset = strchr(param, '+');
 +    dpyname = av_strdup(s1->filename);
 +    offset = strchr(dpyname, '+');
      if (offset) {
          sscanf(offset, "%d,%d", &x_off, &y_off);
          x11grab->nomouse= strstr(offset, "nomouse");
          *offset= 0;
      }
  
-     av_log(s1, AV_LOG_INFO, "device: %s -> display: %s x: %d y: %d width: %d height: %d\n", s1->filename, dpyname, x_off, y_off, ap->width, ap->height);
+     if ((ret = av_parse_video_size(&x11grab->width, &x11grab->height, x11grab->video_size)) < 0) {
+         av_log(s1, AV_LOG_ERROR, "Couldn't parse video size.\n");
+         goto out;
+     }
+ #if FF_API_FORMAT_PARAMETERS
+     if (ap->width > 0)
+         x11grab->width = ap->width;
+     if (ap->height > 0)
+         x11grab->height = ap->height;
+ #endif
+     av_log(s1, AV_LOG_INFO, "device: %s -> display: %s x: %d y: %d width: %d height: %d\n",
 -           s1->filename, param, x_off, y_off, x11grab->width, x11grab->height);
++           s1->filename, dpyname, x_off, y_off, x11grab->width, x11grab->height);
  
 -    dpy = XOpenDisplay(param);
 +    dpy = XOpenDisplay(dpyname);
 +    av_freep(&dpyname);
      if(!dpy) {
          av_log(s1, AV_LOG_ERROR, "Could not open X display.\n");
-         return AVERROR(EIO);
+         ret = AVERROR(EIO);
+         goto out;
      }
  
-     if (ap->width <= 0 || ap->height <= 0 || ap->time_base.den <= 0) {
+     if (ap->time_base.den <= 0) {
          av_log(s1, AV_LOG_ERROR, "AVParameters don't have video size and/or rate. Use -s and -r.\n");
-         return AVERROR(EIO);
+         ret = AVERROR(EINVAL);
+         goto out;
      }
  
      st = av_new_stream(s1, 0);
@@@ -733,11 -723,10 +733,13 @@@ typedef struct AVFormatContext 
  #define AVFMT_FLAG_IGNDTS       0x0008 ///< Ignore DTS on frames that contain both DTS & PTS
  #define AVFMT_FLAG_NOFILLIN     0x0010 ///< Do not infer any values from other values, just return what is stored in the container
  #define AVFMT_FLAG_NOPARSE      0x0020 ///< Do not use AVParsers, you also must set AVFMT_FLAG_NOFILLIN as the fillin code works on frames and no parsing -> no frames. Also seeking to frames can not work if parsing to find frame boundaries has been disabled
- #define AVFMT_FLAG_RTP_HINT     0x0040 ///< Add RTP hinting to the output file
+ #if FF_API_FLAG_RTP_HINT
+ #define AVFMT_FLAG_RTP_HINT     0x0040 ///< Deprecated, use the -movflags rtphint muxer specific AVOption instead
+ #endif
 -
 +#define AVFMT_FLAG_MP4A_LATM    0x0080 ///< Enable RTP MP4A-LATM payload
 +#define AVFMT_FLAG_SORT_DTS    0x10000 ///< try to interleave outputted packets by dts (using this flag can slow demuxing down)
 +#define AVFMT_FLAG_PRIV_OPT    0x20000 ///< Enable use of private options by delaying codec open (this could be made default once all code is converted)
 +#define AVFMT_FLAG_KEEP_SIDE_DATA 0x40000 ///< Dont merge side data but keep it seperate.
      int loop_input;
  
      /**
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
@@@ -245,13 -237,11 +245,13 @@@ static void ff_id3v2_parse(AVFormatCont
              tag[3] = 0;
              tlen = avio_rb24(s->pb);
          }
-         if (tlen > (1<<28))
 -        if (tlen <= 0 || tlen > len - taghdrlen) {
 -            av_log(s, AV_LOG_WARNING, "Invalid size in frame %s, skipping the rest of tag.\n", tag);
++        if (tlen > (1<<28) || !tlen)
              break;
 -        }
          len -= taghdrlen + tlen;
 +
 +        if (len < 0)
 +            break;
 +
          next = avio_tell(s->pb) + tlen;
  
          if (tflags & ID3v2_FLAG_DATALEN) {
Simple merge
Simple merge
Simple merge
Simple merge
@@@ -49,10 -49,9 +49,12 @@@ static const AVOption options[]=
  {"nofillin", "do not fill in missing values that can be exactly calculated", 0, FF_OPT_TYPE_CONST, {.dbl = AVFMT_FLAG_NOFILLIN }, INT_MIN, INT_MAX, D, "fflags"},
  {"noparse", "disable AVParsers, this needs nofillin too", 0, FF_OPT_TYPE_CONST, {.dbl = AVFMT_FLAG_NOPARSE }, INT_MIN, INT_MAX, D, "fflags"},
  {"igndts", "ignore dts", 0, FF_OPT_TYPE_CONST, {.dbl = AVFMT_FLAG_IGNDTS }, INT_MIN, INT_MAX, D, "fflags"},
- {"rtphint", "add rtp hinting", 0, FF_OPT_TYPE_CONST, {.dbl = AVFMT_FLAG_RTP_HINT }, INT_MIN, INT_MAX, E, "fflags"},
+ #if FF_API_FLAG_RTP_HINT
+ {"rtphint", "add rtp hinting (deprecated, use the -movflags rtphint option instead)", 0, FF_OPT_TYPE_CONST, {.dbl = AVFMT_FLAG_RTP_HINT }, INT_MIN, INT_MAX, E, "fflags"},
+ #endif
 +{"sortdts", "try to interleave outputted packets by dts", 0, FF_OPT_TYPE_CONST, {.dbl = AVFMT_FLAG_SORT_DTS }, INT_MIN, INT_MAX, D, "fflags"},
 +{"keepside", "dont merge side data", 0, FF_OPT_TYPE_CONST, {.dbl = AVFMT_FLAG_KEEP_SIDE_DATA }, INT_MIN, INT_MAX, D, "fflags"},
 +{"latm", "enable RTP MP4A-LATM payload", 0, FF_OPT_TYPE_CONST, {.dbl = AVFMT_FLAG_MP4A_LATM }, INT_MIN, INT_MAX, E, "fflags"},
  {"analyzeduration", "how many microseconds are analyzed to estimate duration", OFFSET(max_analyze_duration), FF_OPT_TYPE_INT, {.dbl = 5*AV_TIME_BASE }, 0, INT_MAX, D},
  {"cryptokey", "decryption key", OFFSET(key), FF_OPT_TYPE_BINARY, {.dbl = 0}, 0, 0, D},
  {"indexmem", "max memory used for timestamp index (per stream)", OFFSET(max_index_size), FF_OPT_TYPE_INT, {.dbl = 1<<20 }, 0, INT_MAX, D},
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
@@@ -89,12 -96,10 +96,11 @@@ static int tcp_open(URLContext *h, cons
          ret = connect(fd, cur_ai->ai_addr, cur_ai->ai_addrlen);
      }
  
-     ff_socket_nonblock(fd, 1);
      if (ret < 0) {
 +        int timeout=50;
          struct pollfd p = {fd, POLLOUT, 0};
-         if (ff_neterrno() == AVERROR(EINTR)) {
+         ret = ff_neterrno();
+         if (ret == AVERROR(EINTR)) {
              if (url_interrupt_cb()) {
                  ret = AVERROR_EXIT;
                  goto fail1;
              ret = poll(&p, 1, 100);
              if (ret > 0)
                  break;
 +            if(!--timeout){
 +                av_log(NULL, AV_LOG_ERROR,
 +                    "TCP open %s:%d timeout\n",
 +                    hostname, port);
 +                goto fail;
 +            }
          }
+         if (ret <= 0) {
+             ret = AVERROR(ETIMEDOUT);
+             goto fail;
+         }
          /* test error */
          optlen = sizeof(ret);
          getsockopt (fd, SOL_SOCKET, SO_ERROR, &ret, &optlen);
Simple merge
Simple merge