yuv2rgb icon indicating copy to clipboard operation
yuv2rgb copied to clipboard

[IMP] add bench/bench.c

Open LuoZijun opened this issue 8 years ago • 2 comments

make bench test code is clean then test_yuv_rgb.c.

DONE

  1. ffmpeg_rgb24_to_yuv420p
  2. ffmpeg_yuv420p_to_rgb24
  3. std_rgb24_to_yuv420p

UNDONE

  1. ipp_yuv420p_to_rgb24
  2. ipp_rgb24_to_yuv420p
  3. std_yuv420p_to_rgb24
  4. duration calculations code.

and as your wish, the code indentations all with tabulations (hehe).

the undone things, i will add later.

LuoZijun avatar Dec 09 '16 08:12 LuoZijun

Hey,

This is interesting and seems to be well done, but frankly, i am not sure i will merge it in my repository, even when it is completed, because i want to keep the dependencies to ffmpeg as optionnal as possible, since it can be a PITA, with its continuously changing api, and all the different versions in use.

Actually, i was planning to make ffmpeg dependency completely optionnal (similarly to IPP)

I could probably use your load/save functions, to support more image formats when ffmpeg is enabled, however.

descampsa avatar Dec 10 '16 14:12 descampsa

that's ok, and i agree make ffmpeg is one option.

LuoZijun avatar Dec 12 '16 10:12 LuoZijun