search

mk-fg / python-pulse-control

Python high-level interface and ctypes-based bindings for PulseAudio (libpulse)
https://pypi.org/project/pulsectl/
MIT License
170 stars 36 forks source link

Fix 'pulse_op' call wrapper. #9

Closed bertwesarg closed 8 years ago

bertwesarg commented 8 years ago

One more fix while you have consolidated '_pulse_method_call' in 81e3f5097.

mk-fg commented 8 years ago

Yeah, looks like a bunch of trivial tests with ad-hoc pulse instance could've prevented all that, though of course my bad for not bothering to run the new methods even once. Yet again, thanks for cleaning it up.

bertwesarg commented 8 years ago

You're welcome. Thanks to you for the 'pulsectl' module. Just found it these days and was finally be able to be creative with PA.

mk-fg commented 8 years ago

looks like a bunch of trivial tests with ad-hoc pulse instance could've prevented all that

Hopefully fixed now - https://github.com/mk-fg/python-pulse-control#tests (and by https://github.com/mk-fg/python-pulse-control/blob/b55219e/pulsectl/tests/dummy_instance.py#L120-L159 in particular) Should be easy to run these and see if all basic sanity checks pass.