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:53terry.reedysetrecipients: + terry.reedy, jesstess, Saimadhav.Heblikar
2014-06-11 06:29:53terry.reedysetmessageid: <1402468193.49.0.698777359783.issue21624@psf.upfronthosting.co.za>
2014-06-11 06:29:53terry.reedylinkissue21624 messages
2014-06-11 06:29:52terry.reedycreate