>>>>> snarlydwarf  <[EMAIL PROTECTED]> writes:

> It is easy for me to believe because the proper place to do that is in
> the scanner when the db is built (specifically in MP3.pm).

> That section of code is used only for display, and well after the DB
> has been loaded.

Again, it does not make sense to infer the intent from the current
state of the code implementing this preference.  All the scanning and
db code has been completely rewritten at least once since it was
originally implemented.  I traced the code back in svn history a bit
but at some point I think it was moved to Slim/Schema/Album.pm from
another file but svn log just gives me an error probably because I
have not fixed my working directories since the svn reorg a while
back.

Also, as has been pointed out by others, the proper place to do what
has been requested is not in the scanner, because you don't want to
force a rescan when changing the option.  It's possible the
implementer thought adding some logic in the display code was
sufficient and other bits have been lost in the shuffle.  It might
even have been prior to the introduction of the TRACKARTIST
contributor role which I think changed some things.

Anyway, while I do think SC should support the interpretation of TPE2
which some other programs use, I really don't care enough about this
anymore to keep tracking this thread, as I will happily use the TXX
frames.

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

Reply via email to