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 JayKrish, Todd.Rovito, ezio.melotti, philwebster, python-dev, terry.reedy
Date 2013-08-14.02:41:10
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1376448070.38.0.386273709892.issue18425@psf.upfronthosting.co.za>
In-reply-to
Content
IdleHistory.py is identical in 2.7 and 3.3. It is only imported in PyShell.py (within class PyShell, line 852 in 3.3). History is only initialized once, with the default output_sep. See #18732 for deleting the unneeded parameter and associated cruft.

The pushed patch adds docstrings and a couple of comments and removes most of the redundant prefixes. It does not change any logic.

The attached patch started as JK's patch with names changed and Ezio's changes. All test methods pass. There are three chunks of History.fetch that are not covered. Two require cyclic=False. The other requires special situations with text or cursor changes after one call to fetch but before the next. I will add these soon and push.
History
Date User Action Args
2013-08-14 02:41:10terry.reedysetrecipients: + terry.reedy, ezio.melotti, Todd.Rovito, python-dev, JayKrish, philwebster
2013-08-14 02:41:10terry.reedysetmessageid: <1376448070.38.0.386273709892.issue18425@psf.upfronthosting.co.za>
2013-08-14 02:41:10terry.reedylinkissue18425 messages
2013-08-14 02:41:10terry.reedycreate