[FFmpeg-devel] [PATCH] V210 decoder and encoder

Diego Biurrun diego
Sat May 9 22:40:24 CEST 2009


On Sat, May 09, 2009 at 01:30:40PM -0700, Baptiste Coudurier wrote:
> 
> $subject.

some nits and whatnot

> --- libavcodec/v210.c	(revision 0)
> +++ libavcodec/v210.c	(revision 0)
> @@ -0,0 +1,258 @@
> +
> +    avctx->pix_fmt = PIX_FMT_YUV422P16;
> +    avctx->bits_per_raw_sample = 10;
> +
> +    avctx->coded_frame = avcodec_alloc_frame();

Could be aligned.

> +static int decode_frame(AVCodecContext *avctx, void *data, int *data_size, AVPacket *avpkt)

long line

> +    uint8_t *p = buf;
> +    uint8_t *pdst = buf;

align

> --- libavcodec/allcodecs.c	(revision 18773)
> +++ libavcodec/allcodecs.c	(working copy)
> @@ -45,6 +45,14 @@
>  
> +#ifdef PTW32_STATIC_LIB
> +static void detach_ptw32(void)
> +{
> +    pthread_win32_thread_detach_np();
> +    pthread_win32_process_detach_np();
> +}
> +#endif
> +
> @@ -53,6 +61,12 @@
>  
> +#ifdef PTW32_STATIC_LIB
> +    pthread_win32_process_attach_np();
> +    pthread_win32_thread_attach_np();
> +    atexit(detach_ptw32);
> +#endif

unrelated Windows threading fixes

> @@ -169,6 +183,7 @@
>      REGISTER_DECODER (ULTI, ulti);
>      REGISTER_DECODER (V210X, v210x);
> +    REGISTER_ENCDEC  (V210, v210);

Put this before V210X.

> --- libavcodec/Makefile	(revision 18773)
> +++ libavcodec/Makefile	(working copy)
> @@ -231,6 +231,8 @@
>  OBJS-$(CONFIG_V210X_DECODER)           += v210x.o
> +OBJS-$(CONFIG_V210_DECODER)            += v210.o
> +OBJS-$(CONFIG_V210_ENCODER)            += v210.o

ditto

Diego



More information about the ffmpeg-devel mailing list