|
Barna
Posts: 103
|
 |
« Reply #80 on: 10 Apr '08 - 08:08 » |
|
whoops, it wasn't actually the cover downloader this time, but the reading of covers inside ID3 tags. While starting to use XMPlay API to read inside archives I also changed from opening files with the file system function to XMPlay's file reading (which not only can read local files but also network files). Which resulted in some kind of dead lock while the song was changing as there were two threads in XMPlay trying to open the network file at the same time (XMPlay to play, and my plugin to check if there's an ID3 tag with cover inside). Maybe I could get streamed ID3 covers to work with better timing, but for now I just disabled it completely. Try 0.9.6: Download: http://www.nukular.ch/winamp_svp_0.9.6.zip
|
|
|
|
|
Logged
|
|
|
|
|
amit
Posts: 718
|
 |
« Reply #81 on: 10 Apr '08 - 11:15 » |
|
Hello Barna, Thanks for the fix. Now - whenever a cover has already been downloaded the plugin asks if to replace it (download again). Can you change this behavior ? Isn't it possible to skip download process if a cover is already found in one of the search paths ( tag , track path, default path etc') ?
|
|
|
|
|
Logged
|
|
|
|
|
Barna
Posts: 103
|
 |
« Reply #82 on: 10 Apr '08 - 14:44 » |
|
Ah great I'm an idiot... I didn't only skip loading from ID3 on network streams but loading of covers altogether. Well try updated 0.9.6 (yeah still 0.9.6, too embarassing to note this in a new release) :-) Download: http://www.nukular.ch/winamp_svp_0.9.6.zip
|
|
|
|
|
Logged
|
|
|
|
|
Ian @ un4seen
Administrator
Posts: 15269
|
 |
« Reply #83 on: 10 Apr '08 - 17:22 » |
|
While starting to use XMPlay API to read inside archives I also changed from opening files with the file system function to XMPlay's file reading (which not only can read local files but also network files). Which resulted in some kind of dead lock while the song was changing as there were two threads in XMPlay trying to open the network file at the same time (XMPlay to play, and my plugin to check if there's an ID3 tag with cover inside). Maybe I could get streamed ID3 covers to work with better timing, but for now I just disabled it completely.
Perhaps you could wait for the file to be opened by XMPlay, and check the "Message" info text for the "ID3v2:" header; if that's present, you could then proceed to download and inspect the tags? 
|
|
|
|
|
Logged
|
|
|
|
|
Barna
Posts: 103
|
 |
« Reply #84 on: 10 Apr '08 - 19:42 » |
|
hmm, that does work but it somehow doesn't feel that wise to open a second HTTP connection just to check for an ID3 embedded cover (as it is not that widely used anyway it seems). If I could get the XMPFILE (or a pointer to the memory) of the current streaming song on the other hand :-)
|
|
|
|
|
Logged
|
|
|
|
|
Peekie
Posts: 11
|
 |
« Reply #85 on: 11 Apr '08 - 10:55 » |
|
I've got a bunch of mp3-files with the covers stored inside them. Simply because my iPod can only show album art when it's inside the mp3.
Is the feature of reading the cover from inside the mp3-file going to be available again?
|
|
|
|
|
Logged
|
|
|
|
|
Barna
Posts: 103
|
 |
« Reply #86 on: 11 Apr '08 - 11:02 » |
|
It is still available like it was before... Covers inside MP3 songs (or theoretically other files with ID3v2 headers) loaded from a local file are still being displayed. Only for streamed files (from http or ftp) the cover feature is limited.
|
|
|
|
|
Logged
|
|
|
|
|
Peekie
Posts: 11
|
 |
« Reply #87 on: 11 Apr '08 - 11:25 » |
|
That's weird. I've got the most recent version (0.9.6) loaded and it always displays the Downloading message when a new song starts to play. The picture stored in the mp3 isn't displayed. When I deselect the option to autodownload the cover, it displays the No Album Cover picture. While in an old version (0.6) the stored cover is displayed.
|
|
|
|
|
Logged
|
|
|
|
|
|
|
Ian @ un4seen
Administrator
Posts: 15269
|
 |
« Reply #89 on: 11 Apr '08 - 13:08 » |
|
hmm, that does work but it somehow doesn't feel that wise to open a second HTTP connection just to check for an ID3 embedded cover (as it is not that widely used anyway it seems). If I could get the XMPFILE (or a pointer to the memory) of the current streaming song on the other hand :-)
That would be a little more efficient, but it could also result in problems, eg. if XMPlay unloads the file while you're accessing the memory. Swings and roundabouts  I think it should be fine to just re-open the file; you only need to download the ID3v2 tag, so it shouldn't use up too much extra resources.
|
|
|
|
|
Logged
|
|
|
|
|
Peekie
Posts: 11
|
 |
« Reply #90 on: 11 Apr '08 - 13:13 » |
|
0.9.7 is working fine with covers in ID3-tags. See the picture below. Thanks for the quick fix!
|
|
|
|
Logged
|
|
|
|
|
dypsis
Posts: 85
|
 |
« Reply #91 on: 18 Apr '08 - 00:40 » |
|
I just noticed this plugin. I will test it out later. 
|
|
|
|
|
Logged
|
|
|
|
|
dypsis
Posts: 85
|
 |
« Reply #92 on: 18 Apr '08 - 07:09 » |
|
I couldn't get this to work.
I fail at the last step selecting a Winamp Plugin file.
Do I need Winamp installed?
|
|
|
|
|
Logged
|
|
|
|
|
Barna
Posts: 103
|
 |
« Reply #93 on: 18 Apr '08 - 08:50 » |
|
Well you need at least a Winamp plugin. Some of them (newest MilkDrop, AVS) are packaged withing the main Winamp installation and others come with their own installer requiring to be installed into a directory named "Winamp" or may even require a Winamp installation of some kind.
If you have a plugin file (usually named vis_<something>.dll) make sure you have all required data files/directories and dll dependencies together in one place. For instance MilkDrop2 (vis_milk2.dll) requires nscrt.dll (from the Winamp installation, found in Winamp directory) and its data directory with all presets (found under \Plugins\Milkdrop2 in a Winamp installation).
(Edit: Oh, but of course the cover album art viewing functionality itself doesn't need a Winamp plugin.)
|
|
|
|
« Last Edit: 18 Apr '08 - 08:52 by Barna »
|
Logged
|
|
|
|
|
dypsis
Posts: 85
|
 |
« Reply #94 on: 18 Apr '08 - 09:28 » |
|
I am only looking for the cover art functionality.
But step 4 is my problem. It cannot find anything.
4. Choose "Load Plugin..." and select a winamp plugin file.
|
|
|
|
|
Logged
|
|
|
|
|
Dotpitch
Posts: 2473
|
 |
« Reply #95 on: 18 Apr '08 - 14:14 » |
|
I am only looking for the cover art functionality. The title of this thread explains the functionality of this plugin very well: Winamp Visualisations in XMPlay (0.9.7) (plus album art with auto download). If you want the cover features, you don't have to load a Winamp visual plugin. Tick both 'Show album art' and 'Auto cover download/Enable', and it should work  .
|
|
|
|
|
Logged
|
|
|
|
|
dypsis
Posts: 85
|
 |
« Reply #96 on: 19 Apr '08 - 03:32 » |
|
I am only looking for the cover art functionality. The title of this thread explains the functionality of this plugin very well: Winamp Visualisations in XMPlay (0.9.7) (plus album art with auto download). If you want the cover features, you don't have to load a Winamp visual plugin. Tick both 'Show album art' and 'Auto cover download/Enable', and it should work  . Yes, I know the title explains its functionality. That is why I tried it. But still no luck on step 4. I just get "No Album Cover" displayed even though I have albums covers in my tags.
|
|
|
|
« Last Edit: 19 Apr '08 - 04:21 by dypsis »
|
Logged
|
|
|
|
|
Y@nekŪ
Posts: 236
|
 |
« Reply #97 on: 19 Apr '08 - 10:33 » |
|
I am only looking for the cover art functionality. The title of this thread explains the functionality of this plugin very well: Winamp Visualisations in XMPlay (0.9.7) (plus album art with auto download). If you want the cover features, you don't have to load a Winamp visual plugin. Tick both 'Show album art' and 'Auto cover download/Enable', and it should work  . Yes, I know the title explains its functionality. That is why I tried it. But still no luck on step 4. I just get "No Album Cover" displayed even though I have albums covers in my tags. I think that alternative, default image (user preferred) is better then text "No Album Cover".
|
|
|
|
|
Logged
|
|
|
|
|
Barna
Posts: 103
|
 |
« Reply #98 on: 20 Apr '08 - 17:29 » |
|
But still no luck on step 4. I just get "No Album Cover" displayed even though I have albums covers in my tags. Support for covers in tags is still quite basic. Only plain, non encrypted images in JPEG or PNG format in ID3v2 tags are supported now. Covers in other type of tags are not supported at all (like images stored in the text only Vorbis Comment tags). Other image formats (GIF for instance would be allowed in ID3 tags) are not supported. I only added support for files I had lying around so I didn't spend time to do all kinds of formats and encryption and stuff which no one ends up using in the real world anyway. Means, please post or upload a sample track somewhere so I can look into adding support for it.
|
|
|
|
|
Logged
|
|
|
|
|
dypsis
Posts: 85
|
 |
« Reply #99 on: 20 Apr '08 - 19:40 » |
|
Support for covers in tags is still quite basic. Only plain, non encrypted images in JPEG or PNG format in ID3v2 tags are supported now.
That is what I have used for my tags.
|
|
|
|
|
Logged
|
|
|
|
|