[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):