[FFmpeg-devel] [PATCH] build: restore videotoolbox compilation on iOS

[Clément Bœsch]

On Fri, Oct 02, 2015 at 11:11:37AM +0200, wm4 wrote:
[...]
> > +#if HAVE_UTGETOSTYPEFROMSTRING
> >              vdactx->cv_pix_fmt_type = UTGetOSTypeFromString(pixfmt_str);
> > +#else
> > +            av_log(s, loglevel, "UTGetOSTypeFromString() is not available "
> > +                   "on this platform, %s pixel format can not be honored from "
> > +                   "the command line\n", videotoolbox_pixfmt);
> > +#endif
> >              ret = av_vda_default_init2(s, vdactx);
> >              CFRelease(pixfmt_str);
> >          }
> 
> I'd question the use of this in the first place. What does it do at
> all? I couldn't find documentation for this option at all.
> 
> You could just do a numerical conversion from FourCC or a plain numeric
> value, but I question why this needs to be user-settable at all.

This is a CLI option to save a software convert by making VT outputs
directly in a given pixel format (the NV12 → BGRA can be faster than
swscale for instance).

So the function is just to map from the vt pixel format string (user
specified) to the vt pixel format.

What we can do also is to make a mapping from our pixel format strings to
the VT pixel formats, which could make more sense. But this is a behaviour
change, and I'm just trying to fix the build here.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151002/9e5fe097/attachment.sig>