[ehds]

What problem does this PR solve?

Issue Number: resolve

Problem Summary:

Without std::move, data is passed to Init as an lvalue, In this case, the move constructor will not be called. And for move-only type, this call will not compile.

What is changed and the side effects?

Changed:
std::move(data) is required to move-construct data_mem.

Side effects:

• Performance effects:

• Breaking backward compatibility:

---

Check List:

• Please make sure your changes are compilable.
• When providing us with a new feature, it is best to add related tests.
• Please follow Contributor Covenant Code of Conduct.

[Copilot]

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Copilot review overview

1 open issue

• This test only checks compilation and does not assert runtime behavior. To better cover the… · New

What changed in this PR

This PR fixes MPSCQueue::Enqueue(T&&) so rvalue arguments are actually moved into the node storage, enabling move-only element types and avoiding unintended copies.

Changes:

• Move-construct data_mem by passing std::move(data) to Init in Enqueue(T&&).
• Add a unit test that enqueues a std::unique_ptr<int> to validate move-only support.

File	Description
test/​mpsc_queue_unittest.cc	Adds coverage to ensure move-only types can be enqueued.
src/​butil/​containers/​mpsc_queue.h	Fixes rvalue enqueue to forward as a move into node storage.

_{Review effort: Lite}

[Copilot]

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Copilot review overview

No open issues 🎉

1 resolved since last review

• This test only checks compilation and does not assert runtime behavior. To better cover the…

_{Review effort: Lite}

[chenBright]

LGTM