alexfh added inline comments.
================
Comment at: test/clang-tidy/Inputs/modernize-pass-by-value/header-with-fix.h:1
@@ +1,2 @@
+#include <string>
+
----------------
hokein wrote:
> Usually test should not use STL headers as it relies on the system headers.
>
> You don't have to use std::string to reproduce the crash, IMO.
Yep, just a class with a user-defined move constructor should be fine.
```
// .h file:
struct S {
S(S&&);
S(const S&);
};
struct Foo {
Foo(const S &s);
S s;
};
// .cpp file:
#include "a.h"
Foo::Foo(const S &s) : s(s) {}
```
https://reviews.llvm.org/D23257
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
