Frame types for AVS 3.0

[sh0dan]

Continued from this thread.

@sh0dan

Please don't tell me 0 pitch is forbidden.

No - 0 pitch is no problem. Can't see any problems arising from it.

Regarding your hierarchy - problem is that it's not an easy distinction - is seems ok. But it should also be remembered, that both YV12 and YUY2 are YUV formats - however I still like your distinction better.

So the complete hierarchy could look like this:

General --- IsInterLeaved() --------+--- IsYUV() -- YUY2
         \                           \
          \ IsPlanar()                -- IsRGB() -- RGB24 / RGB32
           |-- IsRGB()
           |__ IsYUV() - IsYV12() - YV12 / I420

I still don't like that you using switch to choose an algorithm - IMO using the functions are both more readable, and safer. For instance, your filter will NOT process I420 material, because the enum is different, for internal purposes that filter writers shouldn't care about.