Use move_from for lambda always

Author: lucic71

cpp2rust/converter/converter_lib.cpp

@@ -1252,8 +1252,7 @@ const clang::LambdaCapture *AsLambdaCapture(const clang::FieldDecl *field) {
 
 bool IsLambdaCopyOrMoveConstructor(const clang::CXXConstructorDecl *ctor) {
   auto decl = ctor->getParent();
-  return ctor->isCopyOrMoveConstructor() && decl->isLambda() &&
-         !decl->captures().empty();
+  return ctor->isCopyOrMoveConstructor() && decl->isLambda();
 }
 
 bool LambdaNeedsDestruction(const clang::CXXRecordDecl *decl) {

tests/unit/lambda_as_field.cpp

@@ -1,6 +1,7 @@
 // ADDITIONAL_COMPILE_FLAGS: -std=c++23
 // no-compile
 #include <assert.h>
+#include <utility>
 
 template <typename F> struct Guard {
   F f;
@@ -16,6 +17,13 @@ template <typename F> struct Holder {
   }
 };
 
+template <typename F> struct Moving {
+  F f;
+  int moves = 0;
+  Moving(F f) : f(f) {}
+  Moving(Moving &&o) : f(std::move(o.f)), moves(o.moves + 1) {}
+};
+
 template <typename T> auto wrap(T fn) {
   return [fn](int x) { return fn(x) + 1; };
 }
@@ -54,5 +62,10 @@ int main() {
   auto ww = wrap(w);
   assert(ww(2) == 202);
 
+  Moving m{[](int x) { return x * 2; }};
+  Moving m2(std::move(m));
+  assert(m2.f(3) == 6);
+  assert(m2.moves == 1);
+
   return 0;
 }