On 2014/12/03 22:00, X Ryl <boite.pour.s...@gmail.com> wrote:
> Please find attached a new patch, with your other remarks fixed.

How is this change related to your patch description?

-#      encoder         "vorbis"                # optional, vorbis or lame
+#      encoder         "vorbis"                # optional, vorbis or lame or 
any encoder (use mpd --verbose to get a list)

This is the very first thing I see in your patch file.

Still no documentation, and still no reason why this feature is
useful.  I don't like it at all, and the commit message needs to
convince me.
mpd-devel mailing list

Reply via email to