Skip to content

Update InterfacePlayerRDK.cpp - #247

Open
nejuma1 wants to merge 1 commit into
feature/federated_2608from
feature/test
Open

Update InterfacePlayerRDK.cpp#247
nejuma1 wants to merge 1 commit into
feature/federated_2608from
feature/test

Conversation

@nejuma1

@nejuma1 nejuma1 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@nejuma1
nejuma1 requested a review from a team as a code owner September 4, 2026 10:22
Copilot AI lite review requested due to automatic review settings September 4, 2026 10:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The added MW_LOG_INFO(...) appears at file scope as an expression statement, which is invalid C++ and is expected to break compilation.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

This PR adds a log statement to InterfacePlayerRDK.cpp.

Changes:

  • Inserted a MW_LOG_INFO(...) call near the top of the file (before CipherToStringMap).
File summaries
File Description
InterfacePlayerRDK.cpp Adds a top-level log statement in the implementation file.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread InterfacePlayerRDK.cpp
#include <assert.h>
#define GST_NORMAL_PLAY_RATE 1

MW_LOG_INFO("This is to test PR");
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