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".