On Mon, Dec 20, 2021 at 2:27 AM Andreas Rheinhardt
<andreas.rheinha...@outlook.com> wrote:
>
> Diederick Niehorster:
> > @@ -545,11 +759,40 @@ dshow_cycle_formats(AVFormatContext *avctx, enum
> > dshowDeviceType devtype,
> > } else {
> > av_log(avctx, AV_LOG_INFO, " pixel_format=%s",
> > av_get_pix_fmt_name(pix_fmt));
> > }
> > - av_log(avctx, AV_LOG_INFO, " min s=%ldx%ld fps=%g max
> > s=%ldx%ld fps=%g\n",
> > + av_log(avctx, AV_LOG_INFO, " min s=%ldx%ld fps=%g max
> > s=%ldx%ld fps=%g",
> > vcaps->MinOutputSize.cx, vcaps->MinOutputSize.cy,
> > 1e7 / vcaps->MaxFrameInterval,
> > vcaps->MaxOutputSize.cx, vcaps->MaxOutputSize.cy,
> > 1e7 / vcaps->MinFrameInterval);
> > + if (extended_format_info) {
> > + enum AVColorRange col_range =
> > dshow_color_range(extended_format_info);
> > + enum AVColorSpace col_space =
> > dshow_color_space(extended_format_info);
> > + enum AVColorPrimaries col_prim =
> > dshow_color_primaries(extended_format_info);
> > + enum AVColorTransferCharacteristic col_trc =
> > dshow_color_trc(extended_format_info);
> > + enum AVChromaLocation chroma_loc =
> > dshow_chroma_loc(extended_format_info);
> > + if (col_range != AVCOL_RANGE_UNSPECIFIED || col_space
> > != AVCOL_SPC_UNSPECIFIED || col_prim != AVCOL_PRI_UNSPECIFIED || col_trc !=
> > AVCOL_TRC_UNSPECIFIED) {
> > + const char *range = av_color_range_name(col_range);
> > + const char *space = av_color_space_name(col_space);
> > + const char *prim =
> > av_color_primaries_name(col_prim);
> > + const char *trc = av_color_transfer_name(col_trc);
> > + av_log(avctx, AV_LOG_INFO, " (%s, %s/%s/%s",
> > + range ? range : "unknown",
> > + space ? space : "unknown",
> > + prim ? prim : "unknown",
> > + trc ? trc : "unknown");
> > + if (chroma_loc != AVCHROMA_LOC_UNSPECIFIED) {
> > + const char *chroma =
> > av_chroma_location_name(chroma_loc);
> > + av_log(avctx, AV_LOG_INFO, ", %s", chroma ?
> > chroma : "unknown");
> > + }
> > + av_log(avctx, AV_LOG_INFO, ")");
> > + }
> > + else if (chroma_loc != AVCHROMA_LOC_UNSPECIFIED) {
> > + const char *chroma =
> > av_chroma_location_name(chroma_loc);
> > + av_log(avctx, AV_LOG_INFO, "(%s)", chroma ? chroma
> > : "unknown");
> > + }
>
> This looks an awful lot like avcodec_string().
The logic is modeled after that indeed, but i only need a small part
of what avcodec_string does, and don't have avcodeccontext here, so
can't use avcodec_string. Is there a change you think i should make
here?
Thanks!
Dee
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
