When I run mpc to get a status update of what's going on, it bothers me that the third line is (usually) so much longer than the others, so I wanted the ability to specify my own format string. What's nice is that I can reuse the formatting code made for -f so it was very easy to do.
I've fixed the issues that you've pointed out, so the code and commit history is much cleaner.
You've made your position clear on getting specific data from mpd, but that isn't reflected in the code: the -f option (song_format.c) extracts specific data from the song, yet one can do this easily through Python as well, so why have it there? I enjoy this feature and want to expand it a bit. This pr does not change existing behavior and I feel that it's quite simple.
When I run mpc to get a status update of what's going on, it bothers me that the third line is (usually) so much longer than the others, so I wanted the ability to specify my own format string. What's nice is that I can reuse the formatting code made for
-f
so it was very easy to do.I've fixed the issues that you've pointed out, so the code and commit history is much cleaner.
You've made your position clear on getting specific data from mpd, but that isn't reflected in the code: the
-f
option (song_format.c
) extracts specific data from the song, yet one can do this easily through Python as well, so why have it there? I enjoy this feature and want to expand it a bit. This pr does not change existing behavior and I feel that it's quite simple.