Use built-in set datatype
authorJoel Rosdahl <joel@rosdahl.net>
Sun, 20 May 2007 14:58:47 +0000 (16:58 +0200)
committerJoel Rosdahl <joel@rosdahl.net>
Sun, 20 May 2007 14:58:47 +0000 (16:58 +0200)
13 files changed:
src/packages/kofoto/alternative.py
src/packages/kofoto/commandline/main.py
src/packages/kofoto/dag.py
src/packages/kofoto/gkofoto/cachingpixbufloader.py
src/packages/kofoto/gkofoto/clipboard.py
src/packages/kofoto/gkofoto/generatehtmldialog.py
src/packages/kofoto/gkofoto/imageversionslist.py
src/packages/kofoto/gkofoto/objectcollection.py
src/packages/kofoto/gkofoto/objectselection.py
src/packages/kofoto/gkofoto/registerimageversionsdialog.py
src/packages/kofoto/outputengine.py
src/packages/kofoto/shelf.py
src/test/shelftests.py

index bddac58..dc3065c 100644 (file)
@@ -2,11 +2,6 @@
 
 __all__ = ["Alternative"]
 
-try:
-    set
-except NameError:
-    from sets import Set as set
-
 class Alternative:
     """A simple variant record (AKA discriminated union) class.
 
index 074647d..3a57509 100755 (executable)
@@ -8,7 +8,6 @@ __all__ = ["main"]
 import codecs
 import getopt
 import os
-from sets import Set
 import sys
 import time
 
@@ -992,7 +991,7 @@ def cmdSearch(env, args):
         for iv in o.getImageVersions():
             ivs.append(iv)
     if env.printIDs:
-        ids = Set()
+        ids = set()
         for iv in ivs:
             ids.add(iv.getImage().getId())
         output = [str(x) for x in ids]
@@ -1313,7 +1312,7 @@ def main(argv):
 
         imgsizesval = env.config.getcoordlist(
             "album generation", "other_image_size_limits")
-        imgsizesset = Set(imgsizesval) # Get rid of duplicates.
+        imgsizesset = set(imgsizesval) # Get rid of duplicates.
         defaultlimit = env.config.getcoordlist(
             "album generation", "default_image_size_limit")[0]
         imgsizesset.add(defaultlimit)
index 27fa8d0..5d30ae0 100644 (file)
@@ -2,7 +2,6 @@
 
 __all__ = ["DAG", "LoopError"]
 
-from sets import Set
 from kofoto.common import KofotoError
 
 class LoopError(KofotoError):
@@ -19,8 +18,8 @@ class DAG:
         initlist -- A list of elements to add to the DAG. The element
                     will be unconnected.
         """
-        self.roots = Set()
-        self.elements = Set()
+        self.roots = set()
+        self.elements = set()
         self.parents = {}
         self.children = {}
         if initlist:
@@ -37,8 +36,8 @@ class DAG:
         """Add an element to the DAG."""
         self.roots.add(element)
         self.elements.add(element)
-        self.parents[element] = Set()
-        self.children[element] = Set()
+        self.parents[element] = set()
+        self.children[element] = set()
 
     def connect(self, parent, child):
         """Add an element to another element."""
@@ -67,7 +66,7 @@ class DAG:
 
         An iterable is returned.
         """
-        visited = Set()
+        visited = set()
         stack = [element]
         while stack:
             el = stack.pop()
@@ -89,7 +88,7 @@ class DAG:
 
         An iterable is returned.
         """
-        visited = Set()
+        visited = set()
         stack = [element]
         while stack:
             el = stack.pop()
index a5b98ac..c68192f 100644 (file)
@@ -5,7 +5,6 @@ __all__ = ["CachingPixbufLoader"]
 import gc
 import os
 import time
-from sets import Set as set
 if __name__ == "__main__":
     import pygtk
     pygtk.require("2.0")
index 0516578..3b275e3 100644 (file)
@@ -1,4 +1,3 @@
-from sets import Set
 from kofoto.shelf import Album, Image
 from kofoto.gkofoto.categories import ClipboardCategories
 
@@ -11,7 +10,7 @@ class Clipboard:
     def __init__(self):
         self.__objects = None
         self.__types = None
-        self.__changedCallbacks = Set()
+        self.__changedCallbacks = set()
         self.clear()
 
     def addChangedCallback(self, callback):
index f85d9c6..89e75ff 100644 (file)
@@ -1,7 +1,6 @@
 import gtk
 import os
 import re
-from sets import Set
 from kofoto.gkofoto.environment import env
 import kofoto.generate
 import kofoto.shelfexceptions
@@ -79,7 +78,7 @@ class GenerateHTMLDialog:
 
         imgsizesval = env.config.getcoordlist(
             "album generation", "other_image_size_limits")
-        imgsizesset = Set(imgsizesval) # Get rid of duplicates.
+        imgsizesset = set(imgsizesval) # Get rid of duplicates.
         defaultlimit = env.config.getcoordlist(
             "album generation", "default_image_size_limit")[0]
         imgsizesset.add(defaultlimit)
index 26e3a0b..f40b8be 100644 (file)
@@ -1,6 +1,5 @@
 import os
 import gtk
-from sets import Set as set
 from kofoto.alternative import Alternative
 from kofoto.shelf import ImageVersionType
 from kofoto.gkofoto.environment import env
index a00c209..42b1807 100644 (file)
@@ -2,7 +2,6 @@ import os
 import gtk
 import gobject
 import gc
-from sets import Set
 from kofoto.shelfexceptions import BadAlbumTagError
 from kofoto.timer import Timer
 from kofoto.gkofoto.environment import env
@@ -27,7 +26,7 @@ class ObjectCollection(object):
         self.__objectSelection = ObjectSelection(self)
         self.__insertionPseudoThread = None
         self.__registeredViews = []
-        self.__disabledFields = Set()
+        self.__disabledFields = set()
         self.__rowInsertedCallbacks = []
         self.__columnsType = [ gobject.TYPE_BOOLEAN,  # COLUMN_VALID_LOCATION
                                gobject.TYPE_BOOLEAN,  # COLUMN_VALID_CHECKSUM
@@ -220,9 +219,9 @@ class ObjectCollection(object):
             else:
                 checkbutton = widgets.get_widget("deleteImageFilesCheckbutton")
                 deleteFiles = checkbutton.get_active()
-            objectIds = Set()
+            objectIds = set()
             # Create a Set to avoid duplicated objects.
-            for obj in Set(self.__objectSelection.getSelectedObjects()):
+            for obj in set(self.__objectSelection.getSelectedObjects()):
                 if deleteFiles and not obj.isAlbum():
                     for iv in obj.getImageVersions():
                         try:
@@ -365,7 +364,7 @@ class ObjectCollection(object):
         env.widgets["statusbarLoadedObjects"].push(1, text)
 
     def _handleNrOfObjectsUpdate(self):
-        updatedDisabledFields = Set()
+        updatedDisabledFields = set()
         if self.__nrOfAlbums == 0:
             updatedDisabledFields.add(u"albumtag")
         if self.__nrOfImages == 0:
index e212538..dbcd76d 100644 (file)
@@ -1,5 +1,4 @@
 from kofoto.gkofoto.environment import env
-from sets import Set
 
 class ObjectSelection:
     def __init__(self, objectCollection):
@@ -13,7 +12,7 @@ class ObjectSelection:
         # This class must know about each object's row to be able to distinguish
         # individual objects in an album that contains multiple instances
         # of the same image or album.
-        self.__changedCallbacks = Set()
+        self.__changedCallbacks = set()
         self.__objectCollection = objectCollection
         self.addChangedCallback(self._nrOfSelectedObjectsChanged)
 
index 4f4dc6e..e64fd33 100644 (file)
@@ -1,6 +1,5 @@
 import os
 import re
-import sets
 
 import gtk
 
@@ -37,7 +36,7 @@ class RegisterImageVersionsDialog:
 
     def run(self, image):
         self._image = image
-        files = sets.Set()
+        files = set()
         for imageversion in image.getImageVersions():
             base, filename = os.path.split(imageversion.getLocation())
             prefix, _ = os.path.splitext(filename)
index 9a3d1e8..bd7f299 100644 (file)
@@ -6,7 +6,6 @@ import codecs
 import os
 import re
 import time
-from sets import Set
 from kofoto.common import symlink_or_copy_file, UnimplementedError
 
 class OutputEngine:
@@ -18,7 +17,7 @@ class OutputEngine:
             'Generated by <a href="http://kofoto.rosdahl.net"'
             ' target="_top">Kofoto</a> %s.' %
             time.strftime("%Y-%m-%d %H:%M:%S"))
-        self.generatedFiles = Set()
+        self.generatedFiles = set()
         self.__dest = None
         self.__imgrefMap = None
 
@@ -194,14 +193,14 @@ class OutputEngine:
         self.env.out("Calculating album paths...\n")
         albummap = _findAlbumPaths(root)
 
-        albumsToGenerate = Set()
+        albumsToGenerate = set()
         if subalbums:
             for subalbum in subalbums:
                 addDescendants(albumsToGenerate, subalbum)
             for subalbum in subalbums:
-                albumsToGenerate |= Set(subalbum.getAlbumParents())
+                albumsToGenerate |= set(subalbum.getAlbumParents())
         else:
-            albumsToGenerate |= Set(albummap.keys())
+            albumsToGenerate |= set(albummap.keys())
 
         self.preGeneration(root)
         i = 1
index 6afd09a..229a1d5 100644 (file)
@@ -18,7 +18,6 @@ import os
 import re
 import threading
 import sqlite as sql
-from sets import Set
 from kofoto.common import UnimplementedError
 from kofoto.dag import DAG, LoopError
 from kofoto.cachedobject import CachedObject
@@ -1458,12 +1457,12 @@ class _Object:
                 " select category from object_category"
                 " where  object = %s",
                 self.getId())
-            self.categories = Set([x[0] for x in cursor])
+            self.categories = set([x[0] for x in cursor])
             self.allCategoriesFetched = True
         if recursive:
-            allcategories = Set()
+            allcategories = set()
             for catid in self.categories:
-                allcategories |= Set(
+                allcategories |= set(
                     self.shelf.categorydag.get().getAncestors(catid))
         else:
             allcategories = self.categories
@@ -1479,7 +1478,7 @@ class _Object:
         self.objid = objid
         self.attributes = {}
         self.allAttributesFetched = False
-        self.categories = Set()
+        self.categories = set()
         self.allCategoriesFetched = False
 
     def _categoriesDirty(self):
index 992335f..d34d4fb 100755 (executable)
@@ -6,7 +6,6 @@ import shutil
 import sys
 import threading
 import unittest
-from sets import Set as set
 
 if __name__ == "__main__":
     cwd = os.getcwd()