Subject: song length database...

From: mld/uade team
Date: Mon Jan 30 20:24:08 2006

To: Kohina-UADE-Forum
Is it just me or is the support for the song length database in the current cvs broken?

---

From: shd
Date: Tue Jan 31 00:36:11 2006

To: Kohina-UADE-Forum
[quote:6b2bd022af="mld/uade team"]Is it just me or is the support for the song length database in the current cvs broken?[/quote:6b2bd022af]

Which tool? uade123 or xmms?

---

From: shd
Date: Tue Jan 31 00:49:58 2006

To: Kohina-UADE-Forum
[quote:7ac0060b82="shd"][quote:7ac0060b82="mld/uade team"]Is it just me or is the support for the song length database in the current cvs broken?[/quote:7ac0060b82]

Which tool? uade123 or xmms?[/quote:7ac0060b82]

Heeh yes. It was broken in the last change. The change series is still incomplete. I'll release new stable when it's done. I'm trying to make it as easy as possibly to re-use code under common/.

---

From: shd
Date: Tue Jan 31 00:51:44 2006

To: Kohina-UADE-Forum
[quote:0c445efcce="shd"][quote:0c445efcce="shd"][quote:0c445efcce="mld/uade team"]Is it just me or is the support for the song length database in the current cvs broken?[/quote:0c445efcce]

Which tool? uade123 or xmms?[/quote:0c445efcce]

Heeh yes. It was broken in the last change. The change series is still incomplete. I'll release new stable when it's done. I'm trying to make it as easy as possibly to re-use code under common/.[/quote:0c445efcce]

The bug in XMMS plugin was that I moved md5sum to struct uade_song, but used the old array that contains md5sum. Here's the fix, I think...

       play_time = (((int64_t) total_bytes) * 1000) / UADE_BYTES_PER_SECOND;
-      if (curmd5[0] != 0)
-       uade_add_playtime(curmd5, play_time, 1);
+      if (uadesong-&gt;md5[0] != 0)
+       uade_add_playtime(uadesong-&gt;md5, play_time, 1);
     }

---

From: mld/uade team
Date: Tue Jan 31 15:50:39 2006

To: Kohina-UADE-Forum
yeah, it works :) Fixed the audacious plugin too.