Message154207
>> - test_glob_to_re() was doing two levels of escaping (r'\' -> r'\\\\')
>> for its expected output when it should only do one (r'\' -> r'\\').
> Fix merged. I don’t fully understand why one place needs two escapes and the others just one.
I agree that this code is confusing, though. Perhaps we should add a
comment in glob_to_re():
# Warning - manipulating a regex with a regex. Here be dragons.
;-) |
|
Date |
User |
Action |
Args |
2012-02-25 08:10:23 | nadeem.vawda | set | recipients:
+ nadeem.vawda, georg.brandl, jaraco, cjw296, benjamin.peterson, tarek, eric.araujo, Adam.Groszer |
2012-02-25 08:10:23 | nadeem.vawda | set | messageid: <1330157423.05.0.337465638202.issue6884@psf.upfronthosting.co.za> |
2012-02-25 08:10:22 | nadeem.vawda | link | issue6884 messages |
2012-02-25 08:10:22 | nadeem.vawda | create | |
|