[FFmpeg-devel] [PATCH]Implement packed 16bit to planar 16bit unscaled rgb converter

[Michael Niedermayer]

On Fri, Jan 10, 2014 at 01:29:34AM +0100, Carl Eugen Hoyos wrote:
> On Thursday 09 January 2014 07:09:37 pm Michael Niedermayer wrote:
> > On Thu, Jan 09, 2014 at 02:24:38PM +0100, Carl Eugen Hoyos wrote:
> > > Hi!
> > >
> > > A user reported an issue when converting r210 to ffv1:
> > > http://ffmpeg.zeranoe.com/forum/viewtopic.php?f=7&t=1731
> > > 
> http://forum.videohelp.com/threads/361133-Lossless-%2810-Bit-RGB-444%29-and-%2810-Bit-YUV-422%29-Compression-Codec-s?p=2290391&viewfull=1#post2290391
> > >
> > > I had sent a patch last year to fix this, iirc there was no usecase for
> > > it.
> > >
> > > Attached is the rebased patch, I only tested one codepath:
> > > $ ffmpeg -i fate-suite/r210/r210.avi -vcodec ffv1 -coder 1 
> > > -pix_fmt gbrp10le out.avi 
> > > $ ffmpeg -i out.avi -vcodec r210 out2.avi
> > > $ ffmpeg -i fate-suite/r210/r210.avi -f framecrc -
> > > $ ffmpeg -i out2.avi -f framecrc -
> > >
> > > The codepath for GBRP*16 is not activated in utils.c, it breaks fate and
> > > is (atm) unrelated.
> > >
> > > Please review, Carl Eugen
> >
> > breaks:
> > ./ffplay -i matrixbench_mpeg2.mpg -vf format=rgb48le,format=gbrp9be
> 
> Thank you, new patch attached.
> 
> Please review, Carl Eugen

seems working

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

During times of universal deceit, telling the truth becomes a
revolutionary act. -- George Orwell
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140110/23d183e8/attachment.asc>