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: The position of special value tables (cmathmodule.c)
Type: Stage:
Components: Extension Modules Versions: Python 2.6
process
Status: closed Resolution: wont fix
Dependencies: Superseder:
Assigned To: mark.dickinson Nosy List: mark.dickinson, ocean-city
Priority: normal Keywords: patch

Created on 2008-04-21 00:42 by ocean-city, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
cmathmodule.patch ocean-city, 2008-04-21 00:42
Messages (4)
msg65657 - (view) Author: Hirokazu Yamamoto (ocean-city) * (Python committer) Date: 2008-04-21 00:42
If putting special value tables near function is preferable,
(as before) probably this patch helps.
msg66483 - (view) Author: Mark Dickinson (mark.dickinson) * (Python committer) Date: 2008-05-09 18:08
I'm actually pretty happy with the tables the way they are;  everything's 
working at the moment, and the current code seems clear enough.

Do you have a strong reason to want this change?
msg66502 - (view) Author: Hirokazu Yamamoto (ocean-city) * (Python committer) Date: 2008-05-09 23:16
Nope, I don't have strong reason. I just thought exceeding 80 line width
is a little ugly, and if data and function are close you can refer both
more easily. Please feel free to close this entry as "WONT FIX".
msg66948 - (view) Author: Mark Dickinson (mark.dickinson) * (Python committer) Date: 2008-05-16 15:07
I agree that having lines >= 80 characters long is ugly (and violates PEP 
7).  But I think this particular case falls under the 'rules are made to 
be broken' comment at the start of PEP 7.

I like the fact that if you squint at the tables hard enough, each one 
looks almost like a copy of the complex plane. :-).
History
Date User Action Args
2022-04-11 14:56:33adminsetgithub: 46916
2008-05-16 15:08:03mark.dickinsonsetstatus: open -> closed
resolution: wont fix
messages: + msg66948
2008-05-09 23:16:15ocean-citysetmessages: + msg66502
2008-05-09 18:08:56mark.dickinsonsetmessages: + msg66483
2008-04-22 02:09:54mark.dickinsonsetpriority: normal
2008-04-21 00:48:07ocean-citysetassignee: mark.dickinson
nosy: + mark.dickinson
2008-04-21 00:42:41ocean-citycreate