Merge remote branch 'qatar/master'
authorMichael Niedermayer <michaelni@gmx.at>
Mon, 9 May 2011 02:27:07 +0000 (04:27 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Mon, 9 May 2011 02:50:56 +0000 (04:50 +0200)
* qatar/master:
  log: Fix an oob array read.
  cosmetics: trim trailing whitespace in postproc
  Ban strncpy() it's too easy to misuse.
  psymodel: Remove wrapper functions.
  aacenc: Replace loop counters in aac_encode_frame() with more descriptive 'ch' and 'w'.
  regtest: remove redundant flags in jpg test
  regtest: use run_ffmpeg in do_image_formats
  regtest: simplify encoding functions
  ffmpeg.c: check for interlaced flag in the correct place.

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
ffmpeg.c
libavcodec/aacenc.c
libavcodec/psymodel.c
libavcodec/psymodel.h
libavfilter/libmpcodecs/mp_image.h
libavutil/internal.h
libavutil/log.c
libpostproc/postprocess.c
tests/codec-regression.sh

diff --cc ffmpeg.c
Simple merge
Simple merge
Simple merge
Simple merge
index 3d566af,0000000..50d3fa1
mode 100644,000000..100644
--- /dev/null
@@@ -1,149 -1,0 +1,150 @@@
 +/*
 + * This file is part of MPlayer.
 + *
 + * MPlayer is free software; you can redistribute it and/or modify
 + * it under the terms of the GNU General Public License as published by
 + * the Free Software Foundation; either version 2 of the License, or
 + * (at your option) any later version.
 + *
 + * MPlayer 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 General Public License for more details.
 + *
 + * You should have received a copy of the GNU General Public License along
 + * with MPlayer; if not, write to the Free Software Foundation, Inc.,
 + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 + */
 +
 +#ifndef MPLAYER_MP_IMAGE_H
 +#define MPLAYER_MP_IMAGE_H
 +
 +#include <stdio.h>
 +#include <stdlib.h>
 +#include <string.h>
 +#undef printf //FIXME
 +#undef fprintf //FIXME
 +#include "mp_msg.h"
 +#include "libavutil/avutil.h"
 +#include "libavutil/avassert.h"
 +#undef realloc
 +#undef malloc
 +#undef free
 +#undef rand
 +#undef srand
 +#undef printf
++#undef strncpy
 +#define ASMALIGN(ZEROBITS) ".p2align " #ZEROBITS "\n\t"
 +
 +
 +//--------- codec's requirements (filled by the codec/vf) ---------
 +
 +//--- buffer content restrictions:
 +// set if buffer content shouldn't be modified:
 +#define MP_IMGFLAG_PRESERVE 0x01
 +// set if buffer content will be READ for next frame's MC: (I/P mpeg frames)
 +#define MP_IMGFLAG_READABLE 0x02
 +
 +//--- buffer width/stride/plane restrictions: (used for direct rendering)
 +// stride _have_to_ be aligned to MB boundary:  [for DR restrictions]
 +#define MP_IMGFLAG_ACCEPT_ALIGNED_STRIDE 0x4
 +// stride should be aligned to MB boundary:     [for buffer allocation]
 +#define MP_IMGFLAG_PREFER_ALIGNED_STRIDE 0x8
 +// codec accept any stride (>=width):
 +#define MP_IMGFLAG_ACCEPT_STRIDE 0x10
 +// codec accept any width (width*bpp=stride -> stride%bpp==0) (>=width):
 +#define MP_IMGFLAG_ACCEPT_WIDTH 0x20
 +//--- for planar formats only:
 +// uses only stride[0], and stride[1]=stride[2]=stride[0]>>mpi->chroma_x_shift
 +#define MP_IMGFLAG_COMMON_STRIDE 0x40
 +// uses only planes[0], and calculates planes[1,2] from width,height,imgfmt
 +#define MP_IMGFLAG_COMMON_PLANE 0x80
 +
 +#define MP_IMGFLAGMASK_RESTRICTIONS 0xFF
 +
 +//--------- color info (filled by mp_image_setfmt() ) -----------
 +// set if number of planes > 1
 +#define MP_IMGFLAG_PLANAR 0x100
 +// set if it's YUV colorspace
 +#define MP_IMGFLAG_YUV 0x200
 +// set if it's swapped (BGR or YVU) plane/byteorder
 +#define MP_IMGFLAG_SWAPPED 0x400
 +// set if you want memory for palette allocated and managed by vf_get_image etc.
 +#define MP_IMGFLAG_RGB_PALETTE 0x800
 +
 +#define MP_IMGFLAGMASK_COLORS 0xF00
 +
 +// codec uses drawing/rendering callbacks (draw_slice()-like thing, DR method 2)
 +// [the codec will set this flag if it supports callbacks, and the vo _may_
 +//  clear it in get_image() if draw_slice() not implemented]
 +#define MP_IMGFLAG_DRAW_CALLBACK 0x1000
 +// set if it's in video buffer/memory: [set by vo/vf's get_image() !!!]
 +#define MP_IMGFLAG_DIRECT 0x2000
 +// set if buffer is allocated (used in destination images):
 +#define MP_IMGFLAG_ALLOCATED 0x4000
 +
 +// buffer type was printed (do NOT set this flag - it's for INTERNAL USE!!!)
 +#define MP_IMGFLAG_TYPE_DISPLAYED 0x8000
 +
 +// codec doesn't support any form of direct rendering - it has own buffer
 +// allocation. so we just export its buffer pointers:
 +#define MP_IMGTYPE_EXPORT 0
 +// codec requires a static WO buffer, but it does only partial updates later:
 +#define MP_IMGTYPE_STATIC 1
 +// codec just needs some WO memory, where it writes/copies the whole frame to:
 +#define MP_IMGTYPE_TEMP 2
 +// I+P type, requires 2+ independent static R/W buffers
 +#define MP_IMGTYPE_IP 3
 +// I+P+B type, requires 2+ independent static R/W and 1+ temp WO buffers
 +#define MP_IMGTYPE_IPB 4
 +// Upper 16 bits give desired buffer number, -1 means get next available
 +#define MP_IMGTYPE_NUMBERED 5
 +// Doesn't need any buffer, incomplete image (probably a first field only)
 +// we need this type to be able to differentiate between half frames and
 +// all other cases
 +#define MP_IMGTYPE_INCOMPLETE 6
 +
 +#define MP_MAX_PLANES 4
 +
 +#define MP_IMGFIELD_ORDERED 0x01
 +#define MP_IMGFIELD_TOP_FIRST 0x02
 +#define MP_IMGFIELD_REPEAT_FIRST 0x04
 +#define MP_IMGFIELD_TOP 0x08
 +#define MP_IMGFIELD_BOTTOM 0x10
 +#define MP_IMGFIELD_INTERLACED 0x20
 +
 +typedef struct mp_image {
 +    unsigned int flags;
 +    unsigned char type;
 +    int number;
 +    unsigned char bpp;  // bits/pixel. NOT depth! for RGB it will be n*8
 +    unsigned int imgfmt;
 +    int width,height;  // stored dimensions
 +    int x,y,w,h;  // visible dimensions
 +    unsigned char* planes[MP_MAX_PLANES];
 +    int stride[MP_MAX_PLANES];
 +    char * qscale;
 +    int qstride;
 +    int pict_type; // 0->unknown, 1->I, 2->P, 3->B
 +    int fields;
 +    int qscale_type; // 0->mpeg1/4/h263, 1->mpeg2
 +    int num_planes;
 +    /* these are only used by planar formats Y,U(Cb),V(Cr) */
 +    int chroma_width;
 +    int chroma_height;
 +    int chroma_x_shift; // horizontal
 +    int chroma_y_shift; // vertical
 +    int usage_count;
 +    /* for private use by filter or vo driver (to store buffer id or dmpi) */
 +    void* priv;
 +} mp_image_t;
 +
 +void mp_image_setfmt(mp_image_t* mpi,unsigned int out_fmt);
 +mp_image_t* new_mp_image(int w,int h);
 +void free_mp_image(mp_image_t* mpi);
 +
 +mp_image_t* alloc_mpi(int w, int h, unsigned long int fmt);
 +void mp_image_alloc_planes(mp_image_t *mpi);
 +void copy_mpi(mp_image_t *dmpi, mp_image_t *mpi);
 +
 +#endif /* MPLAYER_MP_IMAGE_H */
Simple merge
diff --cc libavutil/log.c
Simple merge
Simple merge
@@@ -75,13 -75,8 +75,13 @@@ do_video_encoding msmpeg4.avi "-qscale 
  do_video_decoding
  fi
  
- do_video_encoding msvideo1.avi "" "-an -vcodec msvideo1"
 +if [ -n "$do_msvideo1" ] ; then
++do_video_encoding msvideo1.avi "-an -vcodec msvideo1"
 +do_video_decoding "" "-pix_fmt yuv420p"
 +fi
 +
  if [ -n "$do_wmv1" ] ; then
- do_video_encoding wmv1.avi "-qscale 10" "-an -vcodec wmv1"
+ do_video_encoding wmv1.avi "-qscale 10 -an -vcodec wmv1"
  do_video_decoding
  fi
  
@@@ -248,13 -243,8 +248,13 @@@ do_video_encoding flashsv.flv "-an -vco
  do_video_decoding "" "-pix_fmt yuv420p -sws_flags area+accurate_rnd+bitexact"
  fi
  
- do_video_encoding flashsv2.flv "" "-an -vcodec flashsv2 -sws_flags neighbor+full_chroma_int+accurate_rnd+bitexact -strict experimental"
 +if [ -n "$do_flashsv2" ] ; then
++do_video_encoding flashsv2.flv "-an -vcodec flashsv2 -sws_flags neighbor+full_chroma_int+accurate_rnd+bitexact -strict experimental"
 +#do_video_decoding "" "-pix_fmt yuv420p -sws_flags area+accurate_rnd+bitexact"
 +fi
 +
  if [ -n "$do_roq" ] ; then
- do_video_encoding roqav.roq "" "-vframes 5"
+ do_video_encoding roqav.roq "-vframes 5"
  do_video_decoding "" "-pix_fmt yuv420p"
  fi