bmdFormat8BitBGRA vs. bmdFormat8BitARGB

[lov]

Hi,

for some reason playback prefers bmdFormat8BitARGB instead of bmdFormat8BitBGRA, but I'm not sure this is intentional. In case you are playback OpenGL rendering then bmdFormat8BitBGRA should be preferred since that is the native format of the GPU.

In case bmdFormat8BitARGB is intentional for whatever reason, are you open for receiving a pull request that make it possible to define ARGB or BGRA should be in use?

Thanks!

[oliverbreidenbach]

@lov Thanks for participating!

To be honest, we are currently in discussion about what the correct way is. But please submit the pull request and we'll consider it.

[lov]

For now I just define a "Primary" constant so switching on build time is a little more easier. I like the idea of searching for the first-match of an array of supported pixel formats, we just need to fine a better way to define that array.

I'm not sure what could be the correct way to handle this either. Maybe we could define that as a DeckLinkDevice function like

- (array)primaryPixelFormat { return array[]; }

which allows subclasses to override that method.