odygrd / quill

Asynchronous Low Latency C++ Logging Library
MIT License
1.53k stars 157 forks source link

Multiple definitions of `quill::detail::on_alarm(int)` in `quill/backend/SignalHandler.h` #457

Closed ZijunH closed 4 months ago

ZijunH commented 4 months ago

When including quill in different compilation units, gcc gives the error include/quill/backend/SignalHandler.h:336: multiple definition of 'quill::detail::on_alarm(int)'. Adding inline to the function name fixes the issue.

odygrd commented 4 months ago

thank you for reporting. Fixed in 02119a418c2fa5ed18311f3e2b9ad3f615926a0a