homepage

When saving a source file with non-ascii characters from an IDLE window,
on Windows platform (XP and Server 2003 at least)
with locale English US 
 locale.getdefaultlocale()
 ('en_US', 'cp1252')
IDLE prompts in IOBinding.py with the message 
"non Ascii found, yet no encoding declared, Add a line like # -*- 
coding: cp1252 -*-"
If accepted, the file is saved with the wrong encoding.
Afterwards, it is read back by IDLE without any problem and it looks 
good in the IDLE window.
However, if a string with non-ascii characters is sent to another 
module (i.e. email MIMEText)
the string is wrong.
The same source would be rejected by python in terminal mode and by 
IDLE on Linux.

> If accepted, the file is saved with the wrong encoding
"accept" means click on the "OK" button. You have to click on the 
biggest button "Edit my file" to create a valid file.
The problem here is the text in the dialog but also the possibility to 
create an invalid file. 
Two solutions:
 - don't ask user for a confirmation (always insert the header)
 - replace "OK" button by "Cancel" button: the user wants maybe an 
ASCII only file but inserted by error an unicode character (eg. non 
breaking space!)

Quick but ugly solution: force the #coding: header without asking the 
user for that.

Implementation of the second solution: replace Ok button by a Cancel 
button. Escape key is Cancel, whereas Return key is the default 
action: edit the file.

The entire dialog should go away. The default source encoding is UTF-8
in Python 3.0, and IDLE should know that. The locale's encoding
shouldn't ever be considered.

Here is a patch that removes the entire IDLE coding option machinery,
thus implementing PEP 3120.

The patch is good and does simplify things: the py3k conversion to 
unicode may have some advantages sometimes!
I suggest to also remove the remaining two occurrences of 
"frameEncoding" in configDialog.py. This will suppress the extra space 
between the last two blocks of the dialog.

Thanks for the review. Committed (with the proposed change) as r68022
and r68023.