Apply suggestions from code review

Co-authored-by: Victor Stinner <vstinner@python.org>

Author: vstinner

Lib/json/__init__.py

@@ -275,8 +275,8 @@ def detect_encoding(b):
 
 
 def load(fp, *, cls=None, object_hook=None, parse_float=None,
-        parse_int=None, parse_constant=None, object_pairs_hook=None,
-        array_hook=None, **kw):
+         parse_int=None, parse_constant=None, object_pairs_hook=None,
+         array_hook=None, **kw):
     """Deserialize ``fp`` (a ``.read()``-supporting file-like object containing
     a JSON document) to a Python object.
 
@@ -310,8 +310,8 @@ def load(fp, *, cls=None, object_hook=None, parse_float=None,
 
 
 def loads(s, *, cls=None, object_hook=None, parse_float=None,
-        parse_int=None, parse_constant=None, object_pairs_hook=None,
-        array_hook=None, **kw):
+          parse_int=None, parse_constant=None, object_pairs_hook=None,
+          array_hook=None, **kw):
     """Deserialize ``s`` (a ``str``, ``bytes`` or ``bytearray`` instance
     containing a JSON document) to a Python object.
 

Lib/test/test_json/test_decode.py

@@ -74,6 +74,7 @@ def test_array_hook(self):
         t = self.loads(s, array_hook=tuple)
         self.assertEqual(t, (1, 2, 3))
         self.assertEqual(type(t), tuple)
+
         # Nested array in inner structure with object_hook
         s = '{"xkd": [[1], [2], [3]]}'
         p = frozendict(xkd=((1,), (2,), (3,)))
@@ -83,6 +84,7 @@ def test_array_hook(self):
         self.assertEqual(type(data["xkd"]), tuple)
         for item in data["xkd"]:
             self.assertEqual(type(item), tuple)
+
         self.assertEqual(self.loads('[]', array_hook=tuple), ())
 
     def test_decoder_optimizations(self):

Modules/_json.c

@@ -1270,8 +1270,9 @@ scanner_new(PyTypeObject *type, PyObject *args, PyObject *kwds)
     if (s->object_pairs_hook == NULL)
         goto bail;
     s->array_hook = PyObject_GetAttrString(ctx, "array_hook");
-    if (s->array_hook == NULL)
+    if (s->array_hook == NULL) {
         goto bail;
+    }
     s->parse_float = PyObject_GetAttrString(ctx, "parse_float");
     if (s->parse_float == NULL)
         goto bail;