Guard use of mutex, cv, lock when DUCC0_NO_THREADING is defined.

mreineck / ducc

Fork of https://gitlab.mpcdf.mpg.de/mtr/ducc to simplify external contributions

GNU General Public License v2.0

13 stars 12 forks source link

Guard use of mutex, cv, lock when DUCC0_NO_THREADING is defined. #6

Closed cantonios closed 1 year ago

cantonios commented 1 year ago

For TF, we need them completely disabled. They are disabled in threading.cc and fft.h, so this seems consistent.

mreineck commented 1 year ago

Ouch, that indeed looks like an oversight. Thanks for the patch, will have a closer look tomorrow morning!

mreineck commented 1 year ago

It turns out that I was sloppy with multithreading in other places as well (e.g. in the NUFFT and gridding parts), so I need to fix these too. Still, these places don't affect you right now, so Im going to merge this now and take care of the other issues later!