search

nikhilm / node-taglib

Simple taglib bindings to Javascript using node.js
http://nikhilm.github.com/node-taglib
MIT License
147 stars 25 forks source link

Issues with new async API addition #17

Closed nikhilm closed 12 years ago

nikhilm commented 12 years ago
nikhilm commented 12 years ago

I think I've finally found the answer :) I was testing with taglib v1.7 which is the latest stable release of taglib. Today I built taglib master from source and couldn't trigger the bug after trying a lot, so I went digging through the commit history.

Turns out commit taglib/taglib@0cdfa793e98cab96ab36004c4d59c9b251b4ef9a which uses atomic refcounting on mac & windows was done on master after a 1.7 branch was done and was not included in the 1.7 release. This commit seems to fix the problems. The fix for linux has been done in commit taglib/taglib@223b85d345d4c0ca19f1c1e6e280c83b8543c167, a day after the 1.7 release. Essentially, async support will require the client to use taglib master.