]> granicus.if.org Git - python/commitdiff
bpo-28292: Mark calendar.py helper functions as private. (GH-15113)
authorRaymond Hettinger <rhettinger@users.noreply.github.com>
Sun, 4 Aug 2019 20:14:03 +0000 (13:14 -0700)
committerGitHub <noreply@github.com>
Sun, 4 Aug 2019 20:14:03 +0000 (13:14 -0700)
Lib/calendar.py
Misc/NEWS.d/next/Library/2019-08-04-11-47-58.bpo-28292.vkihH5.rst [new file with mode: 0644]

index 3828c43ed279f8f35715d7ff9d37dedf52252d2d..7550d52c0a94a5eb237358401256d8a794857fdb 100644 (file)
@@ -127,18 +127,18 @@ def monthrange(year, month):
     return day1, ndays
 
 
-def monthlen(year, month):
+def _monthlen(year, month):
     return mdays[month] + (month == February and isleap(year))
 
 
-def prevmonth(year, month):
+def _prevmonth(year, month):
     if month == 1:
         return year-1, 12
     else:
         return year, month-1
 
 
-def nextmonth(year, month):
+def _nextmonth(year, month):
     if month == 12:
         return year+1, 1
     else:
@@ -207,13 +207,13 @@ class Calendar(object):
         day1, ndays = monthrange(year, month)
         days_before = (day1 - self.firstweekday) % 7
         days_after = (self.firstweekday - day1 - ndays) % 7
-        y, m = prevmonth(year, month)
-        end = monthlen(y, m) + 1
+        y, m = _prevmonth(year, month)
+        end = _monthlen(y, m) + 1
         for d in range(end-days_before, end):
             yield y, m, d
         for d in range(1, ndays + 1):
             yield year, month, d
-        y, m = nextmonth(year, month)
+        y, m = _nextmonth(year, month)
         for d in range(1, days_after + 1):
             yield y, m, d
 
diff --git a/Misc/NEWS.d/next/Library/2019-08-04-11-47-58.bpo-28292.vkihH5.rst b/Misc/NEWS.d/next/Library/2019-08-04-11-47-58.bpo-28292.vkihH5.rst
new file mode 100644 (file)
index 0000000..478a1b0
--- /dev/null
@@ -0,0 +1,3 @@
+Mark calendar.py helper functions as being private.  The follows PEP 8
+guidance to maintain the style conventions in the module and it addresses a
+known case of user confusion.