Just get "lead artist" (TPE1), don't try others.
authorbarbieri <barbieri@gmail.com>
Thu, 17 Jan 2008 21:46:14 +0000 (21:46 +0000)
committerbarbieri <barbieri@gmail.com>
Thu, 17 Jan 2008 21:46:14 +0000 (21:46 +0000)
commit330abac98d7004ac8600d0bcc90091751307f7ce
tree950011b1c6cb0076b026d3cf0c7d1b9c5c56f6ca
parentf59efd9a3346486e5c126e56bbc70e5dfdd3d45a
Just get "lead artist" (TPE1), don't try others.

id3lib helpers used to get all of those, using an *ordered* search for
the fields.

However these multiple searches were slow and I'm now doing one walk
and registering the found fields, in the order they're listed on the
file. This lead to "composer" showing before "lead artist" and thus
being used, producing undesired results for "cover albums" where you
want the artist to be the playing one, not the original.

We have 2 solutions, I used the first:
 * just get "lead artist", most of tag editors just edit them or at
   least enable users to change that. Most software just use this.
 * get all the fields, consider them as a group for "todo" analysis
   and at the end of the search use in the specific order. This is
   more complex and would require lots of allocation/copy/free to a
   case that is often uncommon.
lightmediascanner/src/plugins/id3lib/id3lib.cpp