search

hpc / mpifileutils

File utilities designed for scalability and performance.
https://hpc.github.io/mpifileutils
BSD 3-Clause "New" or "Revised" License
164 stars 64 forks source link

valgrind fixes for daos-serialize #490

Closed dsikich closed 3 years ago

dsikich commented 3 years ago

Fixes some valgrind errors in daos-serialize.

Signed-off-by: Danielle Sikich danielle.sikich@intel.com

dsikich commented 3 years ago

Other than the question about some printf calls, this looks good to me. Thanks for running valgrind! We should build that into our LLNL mfu CI when we create one.

@adammoody I took those out. I only had those in there in a test CI run of an updated mpifileutils in DAOS to see if the tests pass now, but I don't want those in master :-) Thanks.