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

List:       tmda-cvs
Subject:    CVS: tmda/TMDA ChangeLog,1.250,1.251 Util.py,1.77,1.78
From:       Timothy Legant <tlegant () users ! sourceforge ! net>
Date:       2003-01-30 3:54:23
[Download RAW message or body]

Update of /cvsroot/tmda/tmda/TMDA
In directory sc8-pr-cvs1:/tmp/cvs-serv28840

Modified Files:
	ChangeLog Util.py 
Log Message:
Fix bug in Util.findmatch where the for loop never ended, regardless of
whether or not a match was found.


Index: ChangeLog
===================================================================
RCS file: /cvsroot/tmda/tmda/TMDA/ChangeLog,v
retrieving revision 1.250
retrieving revision 1.251
diff -u -r1.250 -r1.251
--- ChangeLog	29 Jan 2003 02:25:15 -0000	1.250
+++ ChangeLog	30 Jan 2003 03:54:20 -0000	1.251
@@ -1,3 +1,8 @@
+2003-01-29  Tim Legant  <tim@catseye.net>
+
+	* Util.py (findmatch): Fix bug where the for loop just kept going,
+	regardless of whether a match was found or not.
+
 2003-01-28  Tim Legant  <tim@catseye.net>
 
 	* FilterStore.py: New file.  Part of FilterParser.firstmatch

Index: Util.py
===================================================================
RCS file: /cvsroot/tmda/tmda/TMDA/Util.py,v
retrieving revision 1.77
retrieving revision 1.78
diff -u -r1.77 -r1.78
--- Util.py	29 Jan 2003 02:25:17 -0000	1.77
+++ Util.py	30 Jan 2003 03:54:20 -0000	1.78
@@ -529,7 +529,6 @@
     """Determine whether any of the passed e-mail addresses match a
     Unix shell-style wildcard pattern contained in list.  The
     comparison is case-insensitive."""
-    match = 0
     for address in addrs:
         if address:
             address = string.lower(address)
@@ -549,7 +548,9 @@
                              or fnmatch.fnmatch(address,p2))
                 else:
                     match = fnmatch.fnmatch(address,p)
-    return match
+                if match:
+                    return 1
+    return 0
 
 
 def wraptext(text, column=70, honor_leading_ws=1):

_______________________________________
tmda-cvs mailing list
http://tmda.net/lists/listinfo/tmda-cvs
[prev in list] [next in list] [prev in thread] [next in thread]