[FFmpeg-devel] Extend/optimize RGB to RGB conversions funcs into rgb2rgb.c

[yann.lepetitcorps at free.fr]

I have make a check and computing the gain of my new version of rgb24to32() and
rgb32to24() funcs with 8096 loops of 1024 rgb24->rgba32 and rgba32->rgba24
conversions  :

This isn't a BigEndian plateform

Test original rgb24torgb32() func : 141 ms
Test modified rgb24torgb32() func : 116 ms
Test original rgb32torgb24() func : 121 ms
Test modified rgb32torgb24() func : 93 ms

rgb24to32() : original=141ms modified=116ms (25ms 21.55%)

rgb32to24() : original=121ms modified=93ms (28ms 30.11%)


=> I make the modification into my local source tree and test to generate the
.diff file with git :)


@+
Yannoo

Selon yann.lepetitcorps at free.fr:

> Thanks for your feedback,
>
> I have found a tuto about how to handle git branch/versions and patchs for
> FFMPEG at
> https://bitbucket.org/tran_c/ffmpeg/src/f139838d6473/doc/git-howto.txt
>
> => I begin to use the git/patchs way immediately :)
>
> I have see the #ifdef TEST / main() in a lot of files, where have I to locate
> the #define TEST for to can directly use the test located in the main() entry
> into the modified file ?
>
>
> @+
> Yannoo
>
> Selon Carl Eugen Hoyos <cehoyos at ag.or.at>:
>
> >  <yann.lepetitcorps <at> free.fr> writes:
> >
> > [...]
> >
> > Please consider reading http://www.ffmpeg.org/developer.html -
> > especially "Submitting patches".
> > Alternatively, setup a git clone with your changes.
> >
> > Don't forget to add some performance tests results for
> > patches that change performance critical functions.
> >
> > Carl Eugen
> >
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
>
>
>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: test_rgb2rgba.c
Type: text/x-csrc
Size: 5091 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120909/926d4c0c/attachment.bin>