Skip to content

Comments

Add brcmaudiodecoder detection hook in DeepElementAdded#451

Closed
Copilot wants to merge 5 commits intomasterfrom
copilot/check-audiodecoder-null
Closed

Add brcmaudiodecoder detection hook in DeepElementAdded#451
Copilot wants to merge 5 commits intomasterfrom
copilot/check-audiodecoder-null

Conversation

Copy link

Copilot AI commented Feb 20, 2026

Adds detection for brcmaudiodecoder elements when audio decoder is assigned, providing an extension point for platform-specific handling.

Changes

  • DeepElementAdded.cpp: Check element name prefix after m_curAudioDecoder assignment
  • Follows existing pattern used for brcmaudiosink, amlhalasink in SetupElement.cpp
  • Includes TODO placeholder for future decoder-specific configuration
// Check if audio decoder element name starts with brcmaudiodecoder
if (m_glibWrapper->gStrHasPrefix(GST_ELEMENT_NAME(m_element), "brcmaudiodecoder"))
{
    RIALTO_SERVER_LOG_DEBUG("brcmaudiodecoder detected: %s", m_elementName);
    // TODO: Add brcmaudiodecoder-specific code here
}

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 4 commits February 20, 2026 16:23
Co-authored-by: balasaraswathy-n <199900606+balasaraswathy-n@users.noreply.github.com>
Co-authored-by: balasaraswathy-n <199900606+balasaraswathy-n@users.noreply.github.com>
Co-authored-by: balasaraswathy-n <199900606+balasaraswathy-n@users.noreply.github.com>
Co-authored-by: balasaraswathy-n <199900606+balasaraswathy-n@users.noreply.github.com>
Copilot AI changed the title [WIP] Add check for audio decoder and element name Add brcmaudiodecoder detection hook in DeepElementAdded Feb 20, 2026
@balasaraswathy-n balasaraswathy-n deleted the copilot/check-audiodecoder-null branch February 21, 2026 08:54
@github-actions github-actions bot locked and limited conversation to collaborators Feb 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants