search

r-quantities / units

Measurement units for R
https://r-quantities.github.io/units
173 stars 27 forks source link

Valgrind Issues #315

Closed billdenney closed 2 years ago

billdenney commented 2 years ago

While working in a different package that depends on the units library, we were running valgrind, and we got many errors that were due to libudunits2. I'm not sure, but I'd guess that most of these are related to issues in the parent UDUNITS2 library, but I thought it best to report the issue where I find it. I've attached the file generated when I run the following in Linux:

R -d "valgrind --tool=memcheck --leak-check=full" \
            -e "devtools::load_all();devtools::test()" 2>&1 | \
            tee ../units_test_valgrind.txt

units_test_valgrind.txt

Can you tell if these are issues in the units library or UDUNITS2 or something else?

Enchufa2 commented 2 years ago

This is an upstream issue. Our check results on CRAN are clean (which possibly filter out issues in the shared library).

billdenney commented 2 years ago

Thanks for confirming that it isn't unuts.