Adding support for Theora 4:2:2 and 4:4:4 video files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]<

 



On Sun, Feb 14, 2010 at 03:08:02PM +0100, Giorgio wrote:
> I made all the corrections, I hope it's better now. This is my first
> patch, so thank you for your patience.

I have some more comments, I can of course easily make the modifications
myself, but I thought maybe you're interested in the comments anyway.

> +/* This function converts Theora pixelformat to the corresponding IMGFMT_ */
> +static unsigned int convert_pixelformat(sh_video_t *sh){

I'd change it to
static uint32_t theora_pixelformat2imgfmt(theora_pixelformat fmt)
(so the argument is then context->inf.pixelformat)

> +    switch(context->inf.pixelformat) {
> +       case OC_PF_420: return IMGFMT_YV12;
> +       case OC_PF_422: return IMGFMT_422P;
> +       case OC_PF_444: return IMGFMT_444P;
> +    }

Misses a return for the case that none of these match.
I'd suggest adding a
return 0;


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux