[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdemultimedia/juk
From:       Scott Wheeler <wheeler () kde ! org>
Date:       2005-02-22 23:39:48
Message-ID: 20050222233948.90170ED8A () office ! kde ! org
[Download RAW message or body]

CVS commit by wheeler: 

This line wasn't needed before my kdelibs update tonight, and it should be
possible to get rid of it again, but for now just make extra sure not to
load duplicate items.


  M +6 -0      collectionlist.cpp   1.97


--- kdemultimedia/juk/collectionlist.cpp  #1.96:1.97
@@ -87,4 +87,10 @@ void CollectionList::initialize(Playlist
 PlaylistItem *CollectionList::createItem(const FileHandle &file, QListViewItem *, bool)
 {
+    // It's probably possible to optimize the line below away, but, well, right
+    // now it's more important to not load duplicate items.
+
+    if(m_itemsDict.find(file.absFilePath()))
+        return 0;
+
     PlaylistItem *item = new CollectionListItem(file);
     


[prev in list] [next in list] [prev in thread] [next in thread]