Skip to content

Remove debug printf statements from BluetoothDeviceManager#46

Draft
Copilot wants to merge 20 commits into
developfrom
copilot/sub-pr-35-ac7b40b8-e4fe-4ebe-ac91-8f51445e34b9
Draft

Remove debug printf statements from BluetoothDeviceManager#46
Copilot wants to merge 20 commits into
developfrom
copilot/sub-pr-35-ac7b40b8-e4fe-4ebe-ac91-8f51445e34b9

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 16, 2026

Addresses feedback on #35 to remove debug code before merge.

Changes

  • Removed 11 debug printf statements from BluetoothDeviceManager.cpp - all statements with *** _DEBUG: prefix
  • Added .gitignore to exclude build artifacts (build/, CMakeCache.txt, compiled objects)

Proper logging via LOGERR() macro remains intact for error conditions.

// Before
if (pPersistentStore == nullptr) {
    printf("*** _DEBUG: BluetoothDeviceManager::updateBluetoothDeviceInfoCache: Failed to get PersistentStore interface\n");
    LOGERR("Failed to get PersistentStore interface\n");
    return Core::ERROR_GENERAL;
}

// After
if (pPersistentStore == nullptr) {
    LOGERR("Failed to get PersistentStore interface\n");
    return Core::ERROR_GENERAL;
}

💡 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 3 commits February 16, 2026 16:06
Co-authored-by: pahearn73 <8170037+pahearn73@users.noreply.github.com>
Co-authored-by: pahearn73 <8170037+pahearn73@users.noreply.github.com>
Co-authored-by: pahearn73 <8170037+pahearn73@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix debug code in Bluetooth plug-in enhancements Remove debug printf statements from BluetoothDeviceManager Feb 16, 2026
Copilot AI requested a review from pahearn73 February 16, 2026 16:08
@pahearn73 pahearn73 force-pushed the feature/RDKEMW-13334 branch 2 times, most recently from b395ace to 56a3106 Compare March 3, 2026 16:01
@pahearn73 pahearn73 force-pushed the feature/RDKEMW-13334 branch from 680c735 to c618637 Compare March 5, 2026 03:44
@pahearn73 pahearn73 force-pushed the feature/RDKEMW-13334 branch 3 times, most recently from 4039c7c to 4fcf9e6 Compare March 13, 2026 15:29
Base automatically changed from feature/RDKEMW-13334 to develop March 13, 2026 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants