Message220238
| Author |
terry.reedy |
| Recipients |
Saimadhav.Heblikar, jesstess, terry.reedy |
| Date |
2014年06月11日.06:29:52 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1402468193.49.0.698777359783.issue21624@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
Refinement 1: in doing coverage for UndoDelegator, I noticed that the htest function is counted as missing (uncovered). Both of the following in .coveragerc work to ignore the block: name prefix; comment suffix.
def htest_.*:
.*# htest #
The second is more practical for alphabetical order in htest. It is also less work to change. The particular form marks it as not a normal comment.
Refinement 2: put all imports that are specific to the htest function at the top of the function. Since the function is only called once per process, there is no efficiency consideration. I decided that after the tkinter import at the top got me (wasting time) looking through UndoDelegator for a widget call that might be the source of the leak. The changes can wait until we edit the file anyway, or at least write a test.
However, the docstring at the top of htest.py should be changed. |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2014年06月11日 06:29:53 | terry.reedy | set | recipients:
+ terry.reedy, jesstess, Saimadhav.Heblikar |
| 2014年06月11日 06:29:53 | terry.reedy | set | messageid: <1402468193.49.0.698777359783.issue21624@psf.upfronthosting.co.za> |
| 2014年06月11日 06:29:53 | terry.reedy | link | issue21624 messages |
| 2014年06月11日 06:29:52 | terry.reedy | create |
|