Skip to content

Commit 319cfde

Browse files
committed
decklink: deprecate PsF in/out options
Since now the options can be specified generically by FourCC 'cfpr' (capture) and 'pfpr' (display), there is no need to have this explicitly. Also it doesn't seem to be much useful (at least with the PsF stream from XDCAM, the stream is detected to be interlaced, not progressive, even though decklink reports BMDDeckLinkSupportsAutoSwitchingPPsFOnInput=yes).
1 parent 2603149 commit 319cfde

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

src/video_capture/decklink.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -573,10 +573,6 @@ decklink_help(bool full, const char *query_prop_fcc = nullptr)
573573
col() << "\tIncoming signal should be treated as progressive even if detected as interlaced (PsF).\n";
574574
col() << "\n";
575575

576-
col() << SBOLD("Use1080PsF[=true|false]") << "\n";
577-
col() << "\tIncoming signal should be treated as PsF instead of progressive.\n";
578-
col() << "\n";
579-
580576
col() << SBOLD("nosig-send") << "\n";
581577
col() << "\tSend video even if no signal was detected (useful when video interrupts\n"
582578
"\tbut the video stream needs to be preserved, eg. to keep sync with audio).\n";
@@ -602,6 +598,8 @@ decklink_help(bool full, const char *query_prop_fcc = nullptr)
602598
col() << SBOLD("\taacl[=no]")
603599
<< "\tset analog audio levels to maximum gain "
604600
"(consumer audio level)\n";
601+
col() << SBOLD("\tcfpr[=no]")
602+
<< "\tincoming signal should be treated as PsF instead of progressive\n";
605603
col() << "\n";
606604
} else {
607605
col() << "(other options available, use \"" << SBOLD("fullhelp") << "\" to see complete list of options)\n\n";
@@ -771,6 +769,7 @@ static bool parse_option(struct vidcap_decklink_state *s, const char *opt)
771769
} else if (strcasecmp(opt, "p_not_i") == 0) {
772770
s->p_not_i = true;
773771
} else if (strstr(opt, "Use1080PsF") != nullptr) {
772+
MSG(WARNING, "Use1080PsF deprecated, use 'cfpr[=no] instead\n");
774773
s->device_options[bmdDeckLinkConfigCapture1080pAsPsF].set_flag(strchr(opt, '=') == nullptr || strcasecmp(strchr(opt, '='), "false") != 0);
775774
} else if (strncasecmp(opt, "passthrough", 4) == 0 ||
776775
strncasecmp(opt, "nopassthrough", 6) == 0) {

src/video_display/decklink.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -584,6 +584,8 @@ show_help(bool full, const char *query_prop_fcc = nullptr)
584584
<< "\tset maximum analog audio attenuation (consumer line level)\n";
585585
col() << SBOLD("\t\tvoio=blac|lafa")
586586
<< "\tdisplay either black or last frame when idle\n";
587+
col() << SBOLD("\t\tpfpr[=no]")
588+
<< "\tuse of PsF on output instead of progressive (no for the opposite)\n";
587589

588590
if (!full) {
589591
col() << SBOLD("\tconversion") << "\toutput size conversion, use '-d decklink:fullhelp' for list of conversions\n";
@@ -616,7 +618,6 @@ show_help(bool full, const char *query_prop_fcc = nullptr)
616618
SBOLD("\t\tup1i") << " - simultaneous output of SD and up-converted pollarbox 1080i\n";
617619
col() << SBOLD("\tHDMI3DPacking") << " can be (used in conjunction with \"3D\" option):\n" <<
618620
SBOLD("\t\tSideBySideHalf, LineByLine, TopAndBottom, FramePacking, LeftOnly, RightOnly\n");
619-
col() << SBOLD("\tUse1080PsF[=true|false|keep]") << " flag sets use of PsF on output instead of progressive (default is false)\n";
620621
col() << SBOLD("\tprofile=<P>") << "\tuse desired device profile:\n";
621622
print_bmd_device_profiles("\t\t");
622623
col() << SBOLD("\tconnection=<conn>") << " set output video connection (usually unneeded)\n";
@@ -1283,6 +1284,8 @@ static bool settings_init(struct state_decklink *s, const char *fmt,
12831284
} else if (IS_KEY_PREFIX(ptr, "conversion")) {
12841285
s->device_options[bmdDeckLinkConfigVideoOutputConversionMode].parse(strchr(ptr, '=') + 1);
12851286
} else if (is_prefix_of(ptr, "Use1080pNotPsF") || is_prefix_of(ptr, "Use1080PsF")) {
1287+
MSG(WARNING, "Use1080pNotPsF/Use1080PsF deprecated, "
1288+
"use 'pfpr[=no] instead\n");
12861289
s->device_options[bmdDeckLinkConfigOutput1080pAsPsF].parse(strchr(ptr, '=') + 1);
12871290
if (strncasecmp(ptr, "Use1080pNotPsF", strlen("Use1080pNotPsF")) == 0) { // compat, inverse
12881291
s->device_options[bmdDeckLinkConfigOutput1080pAsPsF].set_flag(s->device_options[bmdDeckLinkConfigOutput1080pAsPsF].get_flag());

0 commit comments

Comments
 (0)