lu-zero / bmdtools

Basic capture and play programs for Blackmagic Design Decklink
Boost Software License 1.0
198 stars 74 forks source link

bmdplay.cpp setting connections #103

Open sbelet opened 5 years ago

sbelet commented 5 years ago

This is may not an issue, more like something odd I'd like to report.

in bmdplay.cpp, lines 535-557

`` //XXX make it generic

switch (connection) {  
case 1:  
    DECKLINK_SET_VIDEO_CONNECTION(bmdVideoConnectionComposite);  
    DECKLINK_SET_AUDIO_CONNECTION(bmdAudioConnectionAnalog);  
    break;  
case 2:  
    DECKLINK_SET_VIDEO_CONNECTION(bmdVideoConnectionComponent);  
    DECKLINK_SET_AUDIO_CONNECTION(bmdAudioConnectionAnalog);  
    break;  
case 3:  
    DECKLINK_SET_VIDEO_CONNECTION(bmdVideoConnectionHDMI);  
    DECKLINK_SET_AUDIO_CONNECTION(bmdAudioConnectionEmbedded);  
    break;  
case 4:  
    DECKLINK_SET_VIDEO_CONNECTION(bmdVideoConnectionSDI);  
    DECKLINK_SET_AUDIO_CONNECTION(bmdAudioConnectionEmbedded);  
    break;  
default:  
    // do not change it  
    break;  
}  

``

The thing is (what I think odd is), same DECKLINK_SET_VIDEO/AUDIO_CONNECTION macro also used in bmdcapture (input). and macro expands to INPUT connections not OUTPUT.

Yes, bmdplay works at least in our cases. I just wanted to report because I don't quite understand whats going on.

Regards.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/71049509-bmdplay-cpp-setting-connections?utm_campaign=plugin&utm_content=tracker%2F256764&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F256764&utm_medium=issues&utm_source=github).