Skip to content

Fix code scanning alerts 46, 47, 48, 49, 51 - #744

Merged
NicolasFussberger merged 4 commits into
mainfrom
copilot/fix-code-scanning-alerts-46-47-48-49-51
Oct 7, 2026
Merged

NicolasFussberger merged 4 commits into
mainfrom
copilot/fix-code-scanning-alerts-46-47-48-49-51

Conversation

Copilot AI commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Fixing some errors coming from ignoring return values of system calls & STL.

Co-authored-by: MaciejKaszynski <161459353+MaciejKaszynski@users.noreply.github.com>
Comment thread externals/ipc_dropin/include/ipc_dropin/ringbuffer.hpp Outdated
Co-authored-by: MaciejKaszynski <161459353+MaciejKaszynski@users.noreply.github.com>
Comment thread score/launch_manager/src/daemon/src/process_group_manager/details/graph.cpp Outdated
@MaciejKaszynski
MaciejKaszynski requested a review from danth October 6, 2026 13:31
@MaciejKaszynski
MaciejKaszynski force-pushed the copilot/fix-code-scanning-alerts-46-47-48-49-51 branch from f1b6326 to ffe0d4f Compare October 6, 2026 13:37

@danth danth left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice!

@MaciejKaszynski
MaciejKaszynski force-pushed the copilot/fix-code-scanning-alerts-46-47-48-49-51 branch from ffe0d4f to 0cc40a8 Compare October 6, 2026 13:50
@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-744: https://eclipse-score.github.io/lifecycle/pr-744/

Comment thread score/launch_manager/src/daemon/src/common/log.hpp
@NicolasFussberger
NicolasFussberger added this pull request to the merge queue Oct 7, 2026
Merged via the queue into main with commit 2c28db9 Oct 7, 2026
20 checks passed
@NicolasFussberger
NicolasFussberger deleted the copilot/fix-code-scanning-alerts-46-47-48-49-51 branch October 7, 2026 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants