Am 11.01.2015, 16:31 Uhr, schrieb Anshul <anshul.ffm...@gmail.com>:


+    result = ctx->dli->EnableVideoInput(ctx->bmd_mode,
+ cctx->v210 ? bmdFormat10BitYUV : bmdFormat8BitYUV,
+                                        bmdVideoInputFlagDefault);

Why not save bmdFormat10BitYUV in cctx instead of 1 or 0 in cctx->v210 flag?

Because this is set directly via AVOption.
Saving the bmdFormat beforehand would require an additional variable and an extra function to map from the BM SDKs internal type.

can you please add in commit message how should I test it. it would be
easier for me.

Ok, an extra patchset with new commit messages follows. Also, I created a patch for the indevs documentation.

Georg
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to