Summary:
Move constructors are just like any other constructors. `this` is not in a valid state. You aren't supposed to destroy `this` before move-constructing it.
This was detected by ubsan stress runs.
UBSAN report:
```
[ RUN ] TemporaryFile.moveCtor
folly/experimental/TestUtil.cpp:91:20: runtime error: load of value 3, which is not a valid value for type 'bool'
[ failed: 1 ]
````
Reviewed By: yfeldblum
Differential Revision:
D6501500
fbshipit-source-id:
83bd0525ac2c1f5951f70f5596034bb6cb0ca985
// Movable, but not copyable
TemporaryFile(TemporaryFile&& other) noexcept {
- reset();
assign(other);
}