[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-dev] RFC: remove TorUtil.Enum



  I have been reviewing the TorCtl package and I will be submitting
tickets (usually with patches) for clean-ups, fixes, etc.  Before I
submit a ticket for removing TorUtil.Enum, I wanted to hear from other
devs.  There are no users of this class within TorCtl, so Enum is now
cruft.  Does anyone have a problem with removing Enum?  If I do not
get negative feedback about this patch, I will submit it later in the
week.

  The following patch is also attached to this message as a gzipped patch file.

diff --git a/TorUtil.py b/TorUtil.py
index 3163d09..1d27144 100644
--- a/TorUtil.py
+++ b/TorUtil.py
@@ -22,7 +22,7 @@ if sys.version_info < (2, 5):
 else:
   from hashlib import sha1

-__all__ = ["Enum", "Enum2", "Callable", "sort_list", "quote",
"escape_dots", "unescape_dots",
+__all__ = ["Enum2", "Callable", "sort_list", "quote", "escape_dots",
"unescape_dots",
       "BufSock", "secret_to_key", "urandom_rng", "s2k_gen",
"s2k_check", "plog",
      "ListenSocket", "zprob", "logfile", "loglevel", "loglevels"]

@@ -127,16 +127,6 @@ def read_config(filename):
   loglevel = config.get('TorCtl', 'loglevel')


-class Enum(object):
-  """ Defines an ordered dense name-to-number 1-1 mapping """
-  def __init__(self, start, names):
-    self.nameOf = {}
-    idx = start
-    for name in names:
-      setattr(self,name,idx)
-      self.nameOf[idx] = name
-      idx += 1
-
 class Enum2(object):
   """ Defines an ordered sparse name-to-number 1-1 mapping """
   def __init__(self, **args):
--
1.7.5.1

-- 
Sean Robinson
WiFi Radar - http://wifi-radar.berlios.de
Python WiFi - http://pythonwifi.wikispot.org
pymnl - http://pymnl.wikispot.org

Attachment: Remove-TorUtil.Enum.patch.gz
Description: GNU Zip compressed data

_______________________________________________
tor-dev mailing list
tor-dev@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-dev