On Tue, Aug 19, 2008 at 8:46 AM, Nick Coghlan <[EMAIL PROTECTED]> wrote: > Brett Cannon wrote: >> On Mon, Aug 18, 2008 at 11:55 AM, Fred Drake <[EMAIL PROTECTED]> wrote: >>> On Aug 18, 2008, at 2:43 PM, Antoine Pitrou wrote: >>>> If it's about silencing warnings, then how about putting it in the >>>> warnings >>>> module? >>> >>> That sounds good to me, and would be very reasonable. This would make a >>> nice context manager. >>> >> >> That works for me as well; just move catch_warning() to 'warnings' and >> leave test.test_support as-is. If we did that, though, I would want to >> make 'record' False by default so that it had more reasonable defaults >> for non-testing purposes. > > Moving this single context manager over to warnings definitely makes > more sense to me than moving the whole test_support module.
Works for me. > It would also be easy enough to add a "catch_warning" factory function > in test_support that switched the default back to record=True so we > didn't have to change all of the invocations in the test suite. -- --Guido van Rossum (home page: http://www.python.org/~guido/) _______________________________________________ Python-Dev mailing list Python-Dev@python.org http://mail.python.org/mailman/listinfo/python-dev Unsubscribe: http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com
