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.

classification
Title: Fix for the bug #679880: 'compile' refuses BOM.
Type: Stage:
Components: Interpreter Core Versions: Python 2.3
process
Status: closed Resolution: accepted
Dependencies: Superseder:
Assigned To: lemburg Nosy List: jvr, kirill_simonov, lemburg, nnorwitz
Priority: normal Keywords: patch

Created on 2003-02-04 21:17 by kirill_simonov, last changed 2022-04-10 16:06 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
tokenizer.c.diff kirill_simonov, 2003-02-04 21:19 A patch for tokenizer.c
a_test_for_the_bom_bug.py kirill_simonov, 2003-02-08 15:59 A test for the BOM bug.
Messages (5)
msg42671 - (view) Author: Kirill Simonov (kirill_simonov) Date: 2003-02-04 21:17
This is a signed char issue.
msg42672 - (view) Author: Neal Norwitz (nnorwitz) * (Python committer) Date: 2003-02-04 21:23
Logged In: YES 
user_id=33168

The patch is small (2 lines) and seems ok.  MAL, if you can
review and it makes sense, I'll test & checkin.
msg42673 - (view) Author: Marc-Andre Lemburg (lemburg) * (Python committer) Date: 2003-02-08 14:32
Logged In: YES 
user_id=38388

Could you also provide a few (unit-)tests for this ?
Thanks.
msg42674 - (view) Author: Kirill Simonov (kirill_simonov) Date: 2003-02-08 16:04
Logged In: YES 
user_id=36553

I have attached a small test that contains
two exec's. The first one is for a file object
and  the second one is for a string.
The second exec fails while it shouldn't.
msg42675 - (view) Author: Just van Rossum (jvr) * (Python triager) Date: 2003-02-09 20:47
Logged In: YES 
user_id=92689

Thanks, I've applied the patch (rev. 2.72 of tokenize.c) and added two little tests to test_builtin.py.
History
Date User Action Args
2022-04-10 16:06:30adminsetgithub: 37908
2003-02-04 21:17:24kirill_simonovcreate