Skip to content

RDKEMW-9964: Removing onStopped GDial notification - #178

Open
yuvaramachandran-gurusamy wants to merge 4 commits into
developfrom
topic/RDKEMW-9964_2
Open

RDKEMW-9964: Removing onStopped GDial notification#178
yuvaramachandran-gurusamy wants to merge 4 commits into
developfrom
topic/RDKEMW-9964_2

Conversation

@yuvaramachandran-gurusamy

Copy link
Copy Markdown
Contributor

RDKEMW-9964: Removing onStopped GDial notification handling

Signed-off-by: yuvaramachandran_gurusamy [yuvaramachandran_gurusamy@comcast.com]

Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com>
Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com>
Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com>
@yuvaramachandran-gurusamy
yuvaramachandran-gurusamy requested a review from a team as a code owner November 11, 2025 14:46
@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

Coverity Issue - Variable copied when it could be moved

"powerState" is passed-by-value as parameter to "std::__cxx11::basic_string<char, std::char_traits, std::allocator >::basic_string(std::__cxx11::basic_string<char, std::char_traits, std::allocator > const &)", when it could be moved instead.

Low Impact, CWE-none
COPY_INSTEAD_OF_MOVE

How to fix

Use "std::move(""powerState"")" instead of "powerState".

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
server/gdialservice.cpp:1219

Comment thread server/gdialservice.cpp Outdated
Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com>
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