[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[minion-cvs] Remove dead code and inaccurate comment



Update of /home/minion/cvsroot/src/minion/lib/mixminion
In directory moria.mit.edu:/tmp/cvs-serv18446/lib/mixminion

Modified Files:
	BuildMessage.py 
Log Message:
Remove dead code and inaccurate comment

Index: BuildMessage.py
===================================================================
RCS file: /home/minion/cvsroot/src/minion/lib/mixminion/BuildMessage.py,v
retrieving revision 1.29
retrieving revision 1.30
diff -u -d -r1.29 -r1.30
--- BuildMessage.py	4 Jan 2003 04:12:50 -0000	1.29
+++ BuildMessage.py	5 Jan 2003 01:27:12 -0000	1.30
@@ -562,13 +562,8 @@
     payload = compressData(payload)
     length = len(payload)
     
-    # FFFF This is an ugly workaround for too-compressable data, so we don't
-    # FFFF create messages that will necessarily be dropped.  We should be
-    # FFFF more sensible on the output side.
     if length > 1024 and length*20 <= origLength:
-        LOG.warn("Double-compressing message so it won't look like a z-bomb")
-        payload = compressData(payload)
-        length = len(payload)
+        LOG.warn("Message is very compressible and will look like a zlib bomb")
 
     paddingLen = PAYLOAD_LEN - SINGLETON_PAYLOAD_OVERHEAD - overhead - length
 
@@ -603,9 +598,7 @@
     contents = payload.getContents()
     # FFFF - We should make this rule configurable.
     maxLen = max(20*1024, 20*len(contents))
-    # FFFF - On encountering an overcompressed piece of data, we should
-    # FFFF   deliver it, still compressed, with a warning -- not merely
-    # FFFF   drop it as a _definite_ bomb.
+
     return uncompressData(payload.getContents(), maxLength=maxLen)
 
 def _checkPayload(payload):