SongPrint: use LightSong::GetDuration()

This properly prints the "Time"/"duration" values for songs in virtual CUE folders. This is loosely related to https://github.com/MusicPlayerDaemon/MPD/issues/1048

SongPrint: use LightSong::GetDuration()
fee282f4 · Max Kellermann · 07d2bc68 · fee282f4 · fee282f4
Commit fee282f4 authored Jan 21, 2021 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 9 additions and 1 deletion

NEWS NEWS +1 -0

SongPrint.cxx src/SongPrint.cxx +8 -1

No files found.
--- a/NEWS
+++ b/NEWS
 ver 0.22.4 (not yet released)
 * protocol
  - fix "readpicture" on 32 bit machines
+  - show duration of songs in virtual playlist (CUE) folders
 * storage
  - curl: fix several WebDAV protocol bugs
 * decoder

--- a/src/SongPrint.cxx
+++ b/src/SongPrint.cxx
@@ -91,7 +91,14 @@ song_print_info(Response &r, const LightSong &song, bool base) noexcept
 	if (song.audio_format.IsDefined())
 		r.Format("Format: %s\n", ToString(song.audio_format).c_str());

-	tag_print(r, song.tag);
+	tag_print_values(r, song.tag);
+
+	const auto duration = song.GetDuration();
+	if (!duration.IsNegative())
+		r.Format("Time: %i\n"
+			 "duration: %1.3f\n",
+			 duration.RoundS(),
+			 duration.ToDoubleS());
 }

 void