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 roger.serwy
Recipients amaury.forgeotdarc, belopolsky, python-dev, roger.serwy, terry.reedy, william.barr
Date 2012-05-27.19:11:07
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1338145868.11.0.718415921296.issue10365@psf.upfronthosting.co.za>
In-reply-to
Content
(I need to write an extension to auto-apply strip-trailing-whitespace on save.)

Terry, I agree that the comment needs improvement. The self.editwin reference to the window needs to be available so that .get_saved() doesn't fail. In retrospect, the original patch obscured this point.

Attached is issue10365_rev3.patch, meant to be used against the latest code. It refactors the shell detection as part of the conditional as well as clarifies the comments.

The "Code for use outside IDLE" is part of a test at the end of the file which uses a duck-typed "EditorWindow" object. This object has flist=None which allows this extra code to run.
History
Date User Action Args
2012-05-27 19:11:08roger.serwysetrecipients: + roger.serwy, terry.reedy, amaury.forgeotdarc, belopolsky, william.barr, python-dev
2012-05-27 19:11:08roger.serwysetmessageid: <1338145868.11.0.718415921296.issue10365@psf.upfronthosting.co.za>
2012-05-27 19:11:07roger.serwylinkissue10365 messages
2012-05-27 19:11:07roger.serwycreate