On 2009/11/02 03:37, kernel error <[email protected]> wrote: > I suggest you a new plugin decoder which is able to read vtx > "chiptune" format (Zx Spectrum music format). I used libayemu > (http://sashnov.nm.ru/libayemu.html), it was released under GPLv2, so > I suppose its compatible with your current license. > Decoder plugin and patch can be found here: > http://git.tuxfamily.org/kernelblog/vtxplugin.git > > As well, I checked some possible errors through valgrind, it founds 3 > errors directly from libayemu but nothing in my code.
Your code looks good, but I don't understand the complicated SNDNEED loop. What's the point of that? Please add some source comments explaining that. Please add the library dependency to the INSTALL file. Your patch misses the required Makefile.am and configure.ac change. And what's your real name, by the way? We use real names for commits in our git repository. You're using git in a funny way. Why not clone our master repository, and create one commit for your plugin? This way, we could just pull your patch with one command. Max ------------------------------------------------------------------------------ Come build with us! The BlackBerry(R) Developer Conference in SF, CA is the only developer event you need to attend this year. Jumpstart your developing skills, take BlackBerry mobile applications to market and stay ahead of the curve. Join us from November 9 - 12, 2009. Register now! http://p.sf.net/sfu/devconference _______________________________________________ Musicpd-dev-team mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/musicpd-dev-team
