sparkslabs / kamaelia

Kamaelia is a toolset that had the aim of making working with concurrency simple and reusable. It is now retired in favour of Guild. (This is the definitive repository for kamaelia however)
Other
12 stars 6 forks source link

Vorbis components need updating for modern Cython/etc #26

Open sparkslabs opened 7 months ago

sparkslabs commented 7 months ago

Quick test shows that the underlying library created to simplify working with vorbis still works as well today as it did 20 years ago, so the only thing that needs updating is the python (cython based - was pyrex based) bindings. For example this works:

~/Development/kamaelia/Code/Python/Bindings/vorbissimple/libvorbissimple/src $ cat  ../../../../Kamaelia/Examples/SupportingMediaFiles/KDE_Startup_2.ogg | ./simple_decoder | aplay --rate=44100 --file-type raw --channels=2  --format=S16_LE -

Whereas this currently fails:

~/Development/kamaelia/Code/Python/Bindings/vorbissimple/python $ cat ../../../Kamaelia/Examples/SupportingMediaFiles/KDE_Startup_2.ogg| ./simple_decoder.py
Segmentation fault (core dumped)

In the grand scheme of things, I think this is pretty good.