|
From: | Riccardo Mottola |
Subject: | Re: [Gap-dev-discuss] Cynthiune loading large playlist |
Date: | Sun, 03 Jun 2012 23:02:26 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux i686; rv:10.0.4) Gecko/20120517 Firefox/10.0.4 Iceape/2.7.4 |
Hi, Philippe Roussel wrote:
Yes, the patch makes sens and it should be commited after the root caouse was found. It helps debugging currently :)Hi, Is that sane, or someone with a better idea how to fix that? The patch looks sane to me but you could do the test in PlaylistController around line 528, there are already similar tests done with FormatTester. It would be nice to find the real cause of the file handle shortage but there are no reason to add unsupported files to the playlist so something like your patch should probably be commited.
Riccardo
[Prev in Thread] | Current Thread | [Next in Thread] |