Michael Herger;208130 Wrote: 
> 
> Looking at your code I see one thing I think should have to be changed:
> 
> caching. Who should do the caching? While MIP does some caching anyway,
> 
> the providing plugin knows much better how long a given value is valid.
> 
> Let's take your example: when does the collection name change? I guess 
> 
> _never_ within a playlist. If MIP checked for every song this would be
> a  
> waste of CPU cycles. I'd therefore rather leave it to the information 
> 
> provider or add a parameter which will tell MIP whether to cache or not
> a  
> value. Eg.
> 
> 0: don't cache
> -1: cache for the song's duration
> > =1: update every x seconds
> 
> What do you think?
> An extra parameter as you describe sounds good. In my situation I think
I will be able to cache it in the plugin, but the extra parameter is
good if other plugins can't cache it.

I also found something that I think is the 2.20 release on my
harddrive, I have attached it to this post if you want it.


+-------------------------------------------------------------------+
|Filename: MusicInfoSCR.pm                                          |
|Download: http://forums.slimdevices.com/attachment.php?attachmentid=2954|
+-------------------------------------------------------------------+

-- 
erland

Erland Isaksson
'My homepage' (http://erland.homeip.net) 'My download page'
(http://erland.homeip.net/download)
(Developer of 'TrackStat, SQLPlayList, DynamicPlayList, Custom Browse,
Custom Scan,  Custom Skip, Multi Library and RandomPlayList plugins'
(http://wiki.erland.homeip.net/index.php/Category:SlimServer))
------------------------------------------------------------------------
erland's Profile: http://forums.slimdevices.com/member.php?userid=3124
View this thread: http://forums.slimdevices.com/showthread.php?t=35997

_______________________________________________
plugins mailing list
[email protected]
http://lists.slimdevices.com/lists/listinfo/plugins

Reply via email to