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 terry.reedy
Recipients taleinat, terry.reedy
Date 2018-06-18.00:50:19
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1529283020.5.0.56676864532.issue33839@psf.upfronthosting.co.za>
In-reply-to
Content
What do you think of changing 'ToolTip' to 'Tooltip'?  The me, the extra 'T' is over-aggressive camel-casing that impedes rather than aids reading.

test_tooltip.py was missing the unittest invocation at the end, so I could not the tests from an editor.  Without it, coverage on my Win 10 was 24%.  With it, 91% (same missing coverage).

There are cases where 100% is close enough to impossible to just move on.  I agree that the misse here are not necessarily showstoppers.  But I like to check whether the failure does not suggest a code change. 

TclError from widget.destroy suggests that we *might* be able to cleanup better.

"The TclError exceptions are reproduced by the htest in tooltip.py. ":
With the code as is?  Or if the try-except is removed?

ValueError: where?

Other comments in the review.
History
Date User Action Args
2018-06-18 00:50:20terry.reedysetrecipients: + terry.reedy, taleinat
2018-06-18 00:50:20terry.reedysetmessageid: <1529283020.5.0.56676864532.issue33839@psf.upfronthosting.co.za>
2018-06-18 00:50:20terry.reedylinkissue33839 messages
2018-06-18 00:50:19terry.reedycreate