This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author belopolsky
Recipients belopolsky, eli.bendersky, terry.reedy
Date 2010-07-26.03:47:21
SpamBayes Score 4.9573123e-06
Marked as misclassified No
Message-id <1280116044.74.0.83617235071.issue9315@psf.upfronthosting.co.za>
In-reply-to
Content
I will defer to your judgement on the level of "whiteness" that is appropriate.

I have passed your patch through 2to3, replaced test_support with support and it looks like we have a test case for a regression in 3.x:

======================================================================
FAIL: test_loop_caller_importing (__main__.TestCallers)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "Lib/test/test_trace.py", line 203, in test_loop_caller_importing
    self.assertEqual(self.tr.results().callers, expected)
AssertionError: {(('/Users/sasha/Work/python-svn/py3k-commit/Lib/trace.py', 'trace', 'runfunc'), [truncated]... != {(('Lib/test/test_trace.py', 'test_trace', '_traced_func_importing'), ('fakefile [truncated]...
Diff is 973 characters long. Set self.maxDiff to None to see it.

----------------------------------------------------------------------

I am attaching a py3k patch.  Let's decide how to proceed.  I would like to commit 3.x and 2.x versions at the same time.   Ideally together with fixes for 3.x bugs.  Another approach would be to temporarily disable failing tests in 3.x, but that would make merging between branches more difficult.

A few more nitpicks:

1. There is no need to use leading or trailing underscores in function or variable names.

2. It would make code more readable if you use longer name for the Trace instance: 'trace' or 'tracer' instead of 'tr'.

3. Your file seems to have white space issues.  Please run "make patchcheck".

4. This may be gratuitous, but I would rather see "func" instead of "foo" and testmodule, testfile, etc. instead of fake*.
History
Date User Action Args
2010-07-26 03:47:24belopolskysetrecipients: + belopolsky, terry.reedy, eli.bendersky
2010-07-26 03:47:24belopolskysetmessageid: <1280116044.74.0.83617235071.issue9315@psf.upfronthosting.co.za>
2010-07-26 03:47:23belopolskylinkissue9315 messages
2010-07-26 03:47:22belopolskycreate