[Python-checkins] gh-91038: Change default argument value to `False` instead of `0` (#31621)

JelleZijlstra webhook-mailer at python.org
Sun Feb 26 21:10:41 EST 2023


https://github.com/python/cpython/commit/f3cb15c88afa2e87067de3c6106664b3f7cd4035
commit: f3cb15c88afa2e87067de3c6106664b3f7cd4035
branch: main
author: Rotzbua <Rotzbua at users.noreply.github.com>
committer: JelleZijlstra <jelle.zijlstra at gmail.com>
date: 2023-02-26T18:10:34-08:00
summary:

gh-91038: Change default argument value to `False` instead of `0` (#31621)

The argument is used as a switch and corresponds to a boolean logic. Therefore it is more intuitive to use the corresponding constant `False` as default value instead of the integer `0`.

Co-authored-by: Shantanu <12621235+hauntsaninja at users.noreply.github.com>
Co-authored-by: Oleg Iarygin <oleg at arhadthedev.net>

files:
A Misc/NEWS.d/next/Library/2023-02-26-12-37-17.gh-issue-91038.S4rFH_.rst
M Doc/library/platform.rst
M Lib/platform.py

diff --git a/Doc/library/platform.rst b/Doc/library/platform.rst
index a0c9f63ab9f9..69c4dfc422c9 100644
--- a/Doc/library/platform.rst
+++ b/Doc/library/platform.rst
@@ -63,7 +63,7 @@ Cross Platform
    string is returned if the value cannot be determined.
 
 
-.. function:: platform(aliased=0, terse=0)
+.. function:: platform(aliased=False, terse=False)
 
    Returns a single string identifying the underlying platform with as much useful
    information as possible.
diff --git a/Lib/platform.py b/Lib/platform.py
index 2dfaf76252db..f2b0d1d1bd3f 100755
--- a/Lib/platform.py
+++ b/Lib/platform.py
@@ -1246,7 +1246,7 @@ def python_compiler():
 
 _platform_cache = {}
 
-def platform(aliased=0, terse=0):
+def platform(aliased=False, terse=False):
 
     """ Returns a single string identifying the underlying platform
         with as much useful information as possible (but no more :).
diff --git a/Misc/NEWS.d/next/Library/2023-02-26-12-37-17.gh-issue-91038.S4rFH_.rst b/Misc/NEWS.d/next/Library/2023-02-26-12-37-17.gh-issue-91038.S4rFH_.rst
new file mode 100644
index 000000000000..2667ff120fd4
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2023-02-26-12-37-17.gh-issue-91038.S4rFH_.rst
@@ -0,0 +1 @@
+:meth:`platform.platform` now has boolean default arguments.



More information about the Python-checkins mailing list