[MPlayer-dev-eng] [PATCH] Additional profile and levels for Xvid encoder and less restricted checking for fps

Carl Eric Codere cecodere at yahoo.ca
Thu Aug 26 17:29:05 CEST 2010


Greetings,
        Sorry for making the patches in one single file, I also fixed the tab problem. Here are the two patches made separately, I am not sure if you already commited nmy patch, but here they are once again in the correct format.

- One patch adds the new profile levels (which have already  been added to the xvid 1.3 baseline)
- The other patches the framerate validation limit in Xvid to make it a warning, since it is allowed to go at certain Xvid profile levels at lower resolution but higher framerates, which would not be permitted before.

Thanks again for taking my patches into consideration, if you could add them to the baseline, if not already done!

Best regards,
Carl 

--- On Thu, 8/26/10, Reimar Döffinger <Reimar.Doeffinger at gmx.de> wrote:

> From: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> Subject: Re: [MPlayer-dev-eng] [PATCH] Additional profile and levels for Xvid encoder and less restricted checking for fps
> To: mplayer-dev-eng at mplayerhq.hu
> Received: Thursday, August 26, 2010, 1:15 AM
> On Fri, Aug 20, 2010 at 02:11:38PM
> -0700, Carl Eric Codere wrote:
> > Greetings,
> >         Included in this
> email is a patch which adds new Simple Profile Levels to the
> Xvid encoder (according to ISO 144496-2 Amendments).
> Furthermore, the frame rate checking should be a warning
> instead of an error, since in certain cases, such as sp3
> (Simple Profile Level 3), encoding at QCIF resolution at 30
> fps is allowed from what I understand from the standard,
> which mencoder did not allow before.
> > 
> > Can you check this patch and include it in the source
> code if it makes sense?
> 
> The indentation for the new profiles is off, and the frame
> rate change should
> be a separate patch.
> Otherwise it seems ok to me.
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ve_xvid4.diff1
Type: application/octet-stream
Size: 1633 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100826/a0c0eba8/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ve_xvid4.diff2
Type: application/octet-stream
Size: 795 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100826/a0c0eba8/attachment-0001.obj>


More information about the MPlayer-dev-eng mailing list