Skip to content

Fix llvm compiler warnings - #701

Merged
MaciejKaszynski merged 1 commit into
eclipse-score:mainfrom
etas-contrib:fix_llvm_warnings
Sep 25, 2026
Merged

MaciejKaszynski merged 1 commit into
eclipse-score:mainfrom
etas-contrib:fix_llvm_warnings

Conversation

@NicolasFussberger

Copy link
Copy Markdown
Contributor
bazel-out/k8-opt-exec-ST-fd11653d5758/bin/score/launch_manager/src/daemon/src/alive_monitor/details/ifexm/_virtual_includes/supervision_handle/score/mw/launch_manager/alive_monitor/details/ifexm/supervision_handle.hpp:42:63: warning: moving a temporary object prevents copy elision [-Wpessimizing-move]
   42 |         : process_id_(process_id), buffer_(buffer), ipc_path_(std::move(internal::aliveInterfacePath(process_id_)))
      |                                                               ^
bazel-out/k8-opt-exec-ST-fd11653d5758/bin/score/launch_manager/src/daemon/src/alive_monitor/details/ifexm/_virtual_includes/supervision_handle/score/mw/launch_manager/alive_monitor/details/ifexm/supervision_handle.hpp:42:63: note: remove std::move call here
   42 |         : process_id_(process_id), buffer_(buffer), ipc_path_(std::move(internal::aliveInterfacePath(process_id_)))
      |                                                               ^~~~~~~~~~                                         ~
2 warnings generated.

score/launch_manager/src/daemon/src/process_group_manager/process_group_manager.cpp:461:68: warning: use of GNU old-style field designator extension [-Wgnu-designator]
  461 |     const bool push_result = event_queue_->push(GetActiveRunTarget{promise : std::move(promise)});
      |                                                                    ^~~~~~~~~
      |                                                                    .promise = 
score/launch_manager/src/daemon/src/process_group_manager/process_group_manager.cpp:517:83: warning: use of GNU old-style field designator extension [-Wgnu-designator]
  517 |     const bool push_result = event_queue_->push(SetRequestedRunTarget{run_target, promise : std::move(promise)});
      |                                                                                   ^~~~~~~~~
      |                                                                                   .promise = 
score/launch_manager/src/daemon/src/process_group_manager/process_group_manager.cpp:517:83: warning: mixture of designated and non-designated initializers in the same initializer list is a C99 extension [-Wc99-designator]
  517 |     const bool push_result = event_queue_->push(SetRequestedRunTarget{run_target, promise : std::move(promise)});
      |                                                                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
score/launch_manager/src/daemon/src/process_group_manager/process_group_manager.cpp:517:71: note: first non-designated initializer is here
  517 |     const bool push_result = event_queue_->push(SetRequestedRunTarget{run_target, promise : std::move(promise)});

@MaciejKaszynski
MaciejKaszynski merged commit 1a49967 into eclipse-score:main Sep 25, 2026
26 of 28 checks passed

This branch was successfully deployed

1 active deployment
workflow-approval — 582e5dd9 Deployed Sep 25, 2026 by NicolasFussberger via Build and test unit-tests-x86_64-qnx / approval #1025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants