__M4RI_HAVE_OPENMP vs HAVE_OPENMP

[Former user]

(This concerns Windows)

Enabling openmp (and making m4ri thread-safe) should be straightforward : set __M4RI_HAVE_OPENMP in m4ri_config.h to 1 and openmp should work...

Unfortunately that is not enough, because there is HAVE_OPENMP.

Just setting __M4RI_HAVE_OPENMP to 1 results in crashes as soon as you try to free the memory, because allocating memory uses said define, while freeing uses HAVE_OPENMP, which is not defined.

[Martin Albrecht]

do not use HAVE_OPENMP, always use M4RI_HAVE_OPENMP (fixes #41)

→ Wiki macro error: Changeset 66bcdee8be11 not found.