[/F]
just for the record, one of those "let's change a lot of code that we don't understand, just because we can"
In the case of try + bare-except, it was more a case of "let's change code we don't understand because it's impossible to guess its intent and that's bad for future maintenance".
things broke the "pre" module in 2.2.
someone changed:
try: repl = pcre_expand(_Dummy, repl) except: m = MatchObject(self, source, 0, end, [])
to
try: repl = pcre_expand(_Dummy, repl) except error: m = MatchObject(self, source, 0, end, [])
but in the most common use case (replacement strings containing group references), the pcre_expand function raises a TypeError exception...
Like I said <wink>. The except clause should list the exceptions it specifically intends to silence, and something as obscure as this case deserves a comment to boot. I also note that if this passed the tests, then the test suite wasn't even trying "the most common use case". there's-more-than-one-kind-of-breakage-illustrated-here-ly y'rs - tim