Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the Bluetooth manager pairing path in btrMgr.c, but currently introduces a temporary/unconditional error log statement in BTRMGR_PairDevice().
Changes:
- Adds an unconditional
BTRMGRLOG_ERRORmessage insideBTRMGR_PairDevice().
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| int j; | ||
| gboolean bIsPS4 = FALSE; | ||
|
|
||
| BTRMGRLOG_ERROR ("Test log to check the coverity failure\n"); |
There was a problem hiding this comment.
This unconditional ERROR-level log looks like temporary debug code ("Test log to check the coverity failure") and will spam logs on every pairing attempt. Please remove it (or gate it behind a debug flag / appropriate condition) before merging.
| BTRMGRLOG_ERROR ("Test log to check the coverity failure\n"); |
| int j; | ||
| gboolean bIsPS4 = FALSE; | ||
|
|
||
| BTRMGRLOG_ERROR ("Test log to check the coverity failure\n"); |
There was a problem hiding this comment.
Indentation on this line uses a tab and does not match the surrounding spacing/indentation style in this file; please align it with adjacent code to keep formatting consistent.
No description provided.