[issue5150] IDLE to support reindent.py

Terry J. Reedy added the comment: reindent.py does the following: 1. Change Python (.py) files to use 4-space indents and no hard tab characters. IDLE editor does this by default, as do other programming editors, except as PEP8 recommends something else for continuation lines within fences

[issue5150] IDLE to support reindent.py

Terry J. Reedy added the comment: Having occasionally forgotten, and had to do whitespace commits, I agree. For Python code, there is never an absolute need for trailing whitespace. Trailing whitespace in string literals can be done by parse time concatenation. '''\ line 1 line 2 ''' 'line

[issue5150] IDLE to support reindent.py

Changes by irdb : -- nosy: +irdb ___ Python tracker ___ ___ Python-bugs-list mailing

[issue5150] IDLE to support reindent.py

Raymond Hettinger added the comment: One other thought, it would be nice if there were an option (on by default) to automatically strip trailing whitespace when a file is saved. That would be much nicer than making the user remember to do periodic clean-ups as they edit. --

[issue5150] IDLE to support reindent.py

Ned Deily added the comment: The code needs to be updated for Python 3 and it needs a test. I'm deassigning it in the hopes that someone will feel free to pick it up. -- assignee: ned.deily - stage: commit review - patch review versions: +Python 3.5 -Python 3.2

[issue5150] IDLE to support reindent.py

Terry J. Reedy added the comment: I use rstrip routinely when editing idlelib files, before committing, so it has been useful for that purpose. I do not happen to know what reindent.py does that the current format options do not. #18704 was about integrating the pep8 style checker. That was

[issue5150] IDLE to support reindent.py

Changes by Andrew Svetlov andrew.svet...@gmail.com: -- nosy: +asvetlov ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5150 ___ ___ Python-bugs-list

[issue5150] IDLE to support reindent.py

Changes by Raymond Hettinger rhettin...@users.sourceforge.net: -- assignee: rhettinger - ned.deily nosy: +ned.deily ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5150 ___

[issue5150] IDLE to support reindent.py

Raymond Hettinger rhettin...@users.sourceforge.net added the comment: Opened back up for the new patch (posted after the previous close). -- status: closed - open ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5150

[issue5150] IDLE to support reindent.py

Éric Araujo mer...@netwok.org added the comment: reindent.py is very much a script: It lacks a nice, full programmatic API, I mean standalone functions to check a file object or a filename and functions implementing the command-line interface. As it is now, you see for example print calls in

[issue5150] IDLE to support reindent.py

Raymond Hettinger rhettin...@users.sourceforge.net added the comment: There's no need to go to python-dev or python-ideas with this one. If someone wants to figure-out a way to add reindent functionality to IDLE, patches are welcome. -- ___ Python

[issue5150] IDLE to support reindent.py

Roger Serwy roger.se...@gmail.com added the comment: I grabbed the core of reindent.py and put it into an extension, unmodified. The original reindent.py will emit Indentation Errors if they exist. ScriptBinding already has nice code to handle these problems by highlighting the error,

[issue5150] IDLE to support reindent.py

Changes by Éric Araujo mer...@netwok.org: -- nosy: +merwok resolution: accepted - fixed stage: needs patch - committed/rejected ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5150 ___

[issue5150] IDLE to support reindent.py

Guilherme Polo ggp...@gmail.com added the comment: This has been closed but why not promote reindent.py to a module and add an option on IDLE to allow a complete reindent.py run ? -- ___ Python tracker rep...@bugs.python.org

[issue5150] IDLE to support reindent.py

Roger Serwy roger.se...@gmail.com added the comment: Here's an extension that adds rstrip() to the Format menu. -- nosy: +serwy Added file: http://bugs.python.org/file14108/RstripExtension.py ___ Python tracker rep...@bugs.python.org

[issue5150] IDLE to support reindent.py

Raymond Hettinger rhettin...@users.sourceforge.net added the comment: Applied in r72999. Will backport to 2.7. -- assignee: kbk - rhettinger resolution: - accepted ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5150

[issue5150] IDLE to support reindent.py

Raymond Hettinger rhettin...@users.sourceforge.net added the comment: Backported in r73001 -- status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5150 ___

[issue5150] IDLE to support reindent.py

Changes by Daniel Diniz aja...@gmail.com: -- nosy: +gpolo stage: - needs patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5150 ___ ___

[issue5150] IDLE to support reindent.py

Terry J. Reedy tjre...@udel.edu added the comment: If the interactive interpreter did not require blank lines within blocks to have at least one 'trailing' space to avoid prematurely ending the block (as when copy from editor window and paste), IDLE could automatically rstrip lines. Given that

[issue5150] IDLE to support reindent.py

New submission from Raymond Hettinger rhettin...@users.sourceforge.net: Add rstrip() to the Format menu. Python's svn repository no longer accepts files with trailing whitespace and it is often necessary to run reindent.py before submitting. It would be nice to have this as a built-in