Issue 633633: Cleanup of test_strptime.py (original) (raw)

Issue633633

➜

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:	Cleanup of test_strptime.py
Type:	Stage:
Components:	Tests	Versions:	Python 2.3

process

Status:	closed	Resolution:	out of date
Dependencies:		Superseder:
Assigned To:	barry	Nosy List:	barry, brett.cannon, gvanrossum, loewis
Priority:	normal	Keywords:	patch

Created on 2002-11-05 04:59 by brett.cannon, last changed 2022-04-10 16:05 by admin. This issue is now closed.

Files
File name	Uploaded	Description	Edit
test_strp_cleanup	brett.cannon,2002-11-05 04:59	diff -u: 2002-11-4

Messages (5)
msg41567 - (view)	Author: Brett Cannon (brett.cannon) *	Date: 2002-11-05 04:59
I finally got around to cleaning up test_strptime.py . Basically all I did was break all the lines that went over 80 characters (although there a few that go over by a char or two). I also removed the __version__ variable. Who ever applies this patch wishes to you can go ahead and also remove the __version__ variable for _strptime.py ; it's a relic and not needed let alone updated since I never remember to. And yes, the testing suite still runs and passes all the tests.
msg41568 - (view)	Author: Barry A. Warsaw (barry) *	Date: 2002-11-12 16:41
Logged In: YES user_id=12800 I'll take this one.
msg41569 - (view)	Author: Guido van Rossum (gvanrossum) *	Date: 2002-11-14 23:27
Logged In: YES user_id=6380 Please go ahead. The style violations in that file have bothered me for a long time. :-(
msg41570 - (view)	Author: Brett Cannon (brett.cannon) *	Date: 2002-11-15 21:23
Logged In: YES user_id=357491 I just started patch #639112 which includes this patch within it since this patch had not been applied yet.
msg41571 - (view)	Author: Martin v. Löwis (loewis) *	Date: 2002-11-27 08:31
Logged In: YES user_id=21627 I've applied #639112 instead.