Fwd: ATF: Stop spreading FUD about other media players

[Jeff Mitchell]

On Friday 21 July 2006 16:51, Ian Monroe wrote:
> Yes we do. :) There's no reason to turn off the read-only features of
> ATF (is there?)

No, not really.  Unless people *like* having to re-add files to the playlist 
et al.  But if people thought that this functionality should be on by default 
(that is, the reading of existing UIDs and using them if they exist) then 
it'd be pretty easy to change that.  Simply get rid of most of the

if( AmarokConfig::advancedTagFeatures() )

tests, and ensure that you always check whether a UID is empty (which should 
be the case everywhere right now unless I missed something).

How do other people feel about making the read-only parts of ATF always on, 
and simply not writing unless they've allowed it?

> Writing statistics to files is silly and ultimately doesn't scale to
> stuff like labels.  So lets not worry about that.

I rather disagree with this.  I'm not sure why you think it is silly, or what 
it has to do with labels, which could take advantage of UIDs but would have 
nothing to do with a separate stats comment.  But from the users that have 
replied to this thread so far they all seemed to be in favor of storing stats 
in files like QL does.

More opinions welcome!

--Jeff