search

mcodev31 / libmsym

molecular point group symmetry lib
MIT License
75 stars 32 forks source link

fix: fix `-Wvla-parameter` #32

Open e-kwsm opened 11 months ago

mcodev31 commented 11 months ago

Hi Eisuke,

It will take me some time to go though all your requests (they generally look good, I just rarely have the time), but this caught my eye since I know a few projects that use c++ compilers w/o vla support, and this looks to me like it would not compile? I'm not currently on a computer where I can check, but the header files need to be c++ compatible.

e-kwsm commented 11 months ago

Thank you for your review. I tried a small snippet and found that g++ 12.2.0 does not support VLA for parameter.