[FFmpeg-devel] [PATCH] Drop unneeded YUV2RGB funcs
Michael Niedermayer
michaelni
Fri Aug 7 13:38:51 CEST 2009
On Fri, Aug 07, 2009 at 09:34:35AM +0300, Kostya wrote:
> On Wed, Aug 05, 2009 at 01:39:49PM +0200, Michael Niedermayer wrote:
> > On Wed, Aug 05, 2009 at 08:51:56AM +0300, Kostya wrote:
> > > On Tue, Aug 04, 2009 at 10:46:57PM -0700, Mike Melanson wrote:
> > > > Kostya wrote:
> > > >> $subj, their functionality is duplicated in YSCALE_YUV_2_ANYRGB_*
> > > >> Also that fixes bug with swapping pixels on odd lines.
> > > >
> > > > Forgot patch.
> > >
> > > Am I a FFmpeg dev or not?
> >
> > no, a real ffmpeg dev would have included benchmarks from several different
> > architectures when he proposes droping some optimized code he wrote that
> > contains a bug that he doesnt want to fix.
>
> s/doesnt want/is too lazy/
understood but it makes no real difference, "too lazy", "too busy", whatever
i prefer ffmpeg to improve slowly over ffmpeg worsening quickly
and if we skip testing code entirely it will worsen quickly
>
> > Yes sure we have a fallback but that may be slower, of course it doesnt
> > have to be slower but as said that has to be checked before droping this
> > code.
>
> I've run "time swscale-example", its execution time seems to increase by
> several percents, so here's proper patch.
id like to note that this isnt really the proper way to do benchmarks but
as the patch fixes a bug instead of removing optimizations it doesnt matter
here
[...]
> yuv2rgb.c | 132 +++++++++++++++++++++++++++++++-------------------------------
> 1 file changed, 66 insertions(+), 66 deletions(-)
> 24dce854c4642161db855f18df3e1848600aff10 swscale-yuv2rgb.patch
if the code is tested and works iam fine with it
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090807/70128d1c/attachment.pgp>
More information about the ffmpeg-devel
mailing list