Open UnkDevE opened 4 years ago
Thanks!
On Fri, 10 Dec 2021, 09:26 Jonathan Wakely, @.***> wrote:
@.**** requested changes on this pull request.
This looks wrong, the variables should be extern and defined in exactly one .c file. #185 https://github.com/jarcode-foss/glava/pull/185 does it right.
For why a fix is needed for GCC 10, see the changes for -fno-common in the GCC 10 release notes https://gcc.gnu.org/gcc-10/changes.html#c and Porting to GCC 10 https://gcc.gnu.org/gcc-10/porting_to.html#common page.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jarcode-foss/glava/pull/171#pullrequestreview-828641889, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACS4PRRKKQ5ARCCUT4ZF6CLUQHBVRANCNFSM4P2KRSZA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
While compiling from source, I ran into a few errors,
some header files were missing from glfft and there was a multiple definitions error with the function pointers in glava.h.
adding the headers missing and making the function pointers static fix the problem.
However there are warnings from the function pointers, so I'm unsure whether using static was the right option.
Here are the warnings: