Matt Liberty

Results 1724 comments of 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...

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.