Apply suggestions from code review

Co-authored-by: Paul Ganssle <[email protected]>

Author: donBarbos

Lib/_pydatetime.py

@@ -502,6 +502,8 @@ def _parse_isoformat_time(tstr):
             tzsign = -1 if tstr[tz_pos - 1] == '-' else 1
 
             try:
+                # This function is intended to validate datetimes, but because
+                # we restrict time zones to ±24h, it serves here as well.
                 _check_time_fields(hour=tz_comps[0], minute=tz_comps[1],
                                    second=tz_comps[2], microsecond=tz_comps[3],
                                    fold=0)
@@ -1647,7 +1649,7 @@ def fromisoformat(cls, time_string):
                 raise error_from_tz
             if error_from_components:
                 raise ValueError(
-                    "minute, second, and microsecond must be 0 when hour is 24"
+                    "Minute, second, and microsecond must be 0 when hour is 24"
                 )
 
             return cls(*time_components)

Lib/test/datetimetester.py

@@ -3569,6 +3569,8 @@ def test_fromisoformat_fails_datetime(self):
             '2009-04-19T12:30:45. 400',        # Space before fraction (gh-130959)
             '2009-04-19T12:30:45+00:90:00', # Time zone field out from range
             '2009-04-19T12:30:45+00:00:90', # Time zone field out from range
+            '2009-04-19T12:30:45-00:90:00', # Time zone field out from range
+            '2009-04-19T12:30:45-00:00:90', # Time zone field out from range
         ]
 
         for bad_str in bad_strs:

Modules/_datetimemodule.c

@@ -1135,7 +1135,7 @@ parse_isoformat_time(const char *dtstr, size_t dtlen, int *hour, int *minute,
     rv = parse_hh_mm_ss_ff(tzinfo_pos, p_end, &tzhour, &tzminute, &tzsecond,
                            tzmicrosecond);
 
-    // Check if Timezone fields are in range
+    // Check if timezone fields are in range
     if (check_time_args(tzhour, tzminute, tzsecond, *tzmicrosecond, 0) < 0) {
         return -6;
     }