|
From: | Petteri Hintsanen |
Subject: | Re: [emms-help] 4.2 release tasks: emms-print-metadata install |
Date: | Sat, 10 Dec 2016 18:02:43 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.4.0 |
On 25.10.2016 16:52, Rasmus wrote:
Yoni Rabkin <address@hidden> writes:OK, so there is a good reason to move to C++ with this, which would otherwise seem like overkill to me. Since we would want to test it a bit before packing it up into a release it can hopefully go out with the release after this one (4.3, which is scheduled for May 2017).That should be fine. I don't know what the status is of this ATM.
Hello and apologies for not responding in a timely manner. For some weird reason I had "dropped" EMMS folder in my own mail setup, so these messages didn't catch me until now.
The patch is ok. It's a really simple piece of C++98 so it ought to work with almost any compiler. I haven't submitted the patch earlier because the docs were missing. But I have updated the texinfo and the man page, so I'll post the patch Real Soon Now.
Some relevant copy-paste from correspondence with Rasmus last February:"Btw, there is also a (feature-wise) limited Perl implementation in emms/src. emms.texi does not mention anything about it. The program uses Audio::Scan which seems to be yet another C-based parser for many open media formats. I think we should either drop it or at least document it."
regards, Petteri
[Prev in Thread] | Current Thread | [Next in Thread] |