[issue4285] Use a named tuple for sys.version_info
Ross Light added the comment: Hello, my name is Ross Light. I've written a patch for this, but this is my first patch, so someone please review. This does pass all regression tests, but I did have to modify the test_sys case to not check for sys.version_info being a tuple. -- keywords: +patch nosy: +ross.light Added file: http://bugs.python.org/file12854/patch-4285a.diff ___ Python tracker <http://bugs.python.org/issue4285> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue4285] Use a named tuple for sys.version_info
Ross Light added the comment: Oh yes, you're right. Sorry! Added file: http://bugs.python.org/file12857/patch-4285b.diff ___ Python tracker <http://bugs.python.org/issue4285> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue4285] Use a named tuple for sys.version_info
Ross Light added the comment: Okay, here's a patch with the requested changes. You're right in saying that flag is usually boolean, I was just going along with several other files where there's float and int flags (i.e. floatobject.c). Added file: http://bugs.python.org/file12860/patch-4285c.diff ___ Python tracker <http://bugs.python.org/issue4285> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue4285] Use a named tuple for sys.version_info
Ross Light added the comment: Tests added and new patch uploaded. Anything else, anyone? Added file: http://bugs.python.org/file12870/patch-4285d.diff ___ Python tracker <http://bugs.python.org/issue4285> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
