fossfreedom/coverart-browser — pull request #1 by asermax (2012-09-07 22:45:19+00:00)
Title: Catched exception in album_load. Unsupported image formats would stop the routine

The routine would stop at some point when it found a unsupported image format on my music directory. I just added a try;except to catch the exception and ignore it.

On a side note:
I really like how the plugin looks and I can imagine all the functionality that can be build over it. Neverthless, I think after porting (or at the same time, why not), the code should get a major refactoring, for various reasons:
- The code, as it is, is pretty hard to maintain or extend. It's understandable, but it's harder to handle than it should.
- The performance while loading the covers is pretty poor. In my case, it would take around 30 segs to populate the view, and it flickered constantly until the routine finished. A better approach would be to load only the covers of albums shown on the viewport, for example.

I get that probably your first intention is to port it; in any case, if you-- we(coz I'm interested on it too d:) intend to fix some stuff and add functionlity, a little effort refactoring it would probably save us some headaches on the future.

Anyway, great plugin foss, nice going :3