Matt Liberty
Matt Liberty
Please also note you are missing DCO and need to run clang-format before the merge.
I already made the same suggestion above: `std::tmpnam or std::tmpfile should be used.`
POSIX has tempnam (not tmpnam) which allows a directory specification if that is helpful.
> POSIX has tempnam (not tmpnam) which allows a directory specification if that is helpful. You can give directory and file name prefix which could be ".odb". It seems like...
Thanks for contributing!
The nit picking can migrate to #5181
I can't pick out a clear issue from that. Does it make sense to you?
@QuantamHD waiting on your PR for TSAN
Where does this stand with CI/metrics? Are any updates needed?
No more info is needed for now. This week will be slow as several people at ETH Zurich.