useless code in QueueManager

Erik Hovland erik at hovland.org
Fri Feb 2 00:40:33 UTC 2007


In the member function removeQueuedItem() there is an if block that
searches a list view for a list view item. But the recovered list view
item is never used in any useful way. So it is a waste of cpu cycles
(not that there isn't enough).

I opened a bug (141038) and submitted a patch against it for your
approval.

This patch is applyable to both trunk and the stable branch.

Please review and approve for the upcoming release.

Thanks

E

-- 
Erik Hovland
mail: erik at hovland.org
web: http://hovland.org/
PGP/GPG public key available on request




More information about the Amarok mailing list