vidify / old-audiosync

First implementation of the audio synchronization feature for Vidify, now obsolete
GNU Lesser General Public License v3.0
17 stars 3 forks source link

Specifying a minimum C version #54

Closed marioortizmanero closed 4 years ago

marioortizmanero commented 4 years ago

Currently this extension doesn't specify a minimum C standard. I think using C11 would be best.

It should also include a clear definition of optional features required to compile, such as threads.h or stdatomic.h.

Turns out the threading and atomic part of C11 are optional, and hence not really supported everywhere. Cross platform support (#53) using C might not be the best idea after all. Using C++ is something to consider, too. That, or requiring something like libuv. Not sure.

marioortizmanero commented 4 years ago

Closing issue, see: https://github.com/vidify/audiosync-rs/issues/2