use PyUnicdodeWriter_Format and if statement braces

Author: ynir3

Objects/exceptions.c

@@ -1871,24 +1871,32 @@ ImportError_repr(PyObject *self)
     PyObject *r = BaseException_repr(self);
     PyImportErrorObject *exc = PyImportErrorObject_CAST(self);
     PyUnicodeWriter *writer = PyUnicodeWriter_Create(0);
-    if (writer == NULL) goto error;
-    if (PyUnicodeWriter_WriteSubstring(writer, r, 0, PyUnicode_GET_LENGTH(r)-1) < 0) goto error;
+    if (writer == NULL) {
+        goto error;
+    }
+    if (PyUnicodeWriter_WriteSubstring(writer, r, 0, PyUnicode_GET_LENGTH(r)-1) < 0) {
+        goto error;
+    }
     if (exc->name) {
         if (hasargs) {
-            if (PyUnicodeWriter_WriteASCII(writer, ", ", 2) < 0) goto error;
+            if (PyUnicodeWriter_WriteASCII(writer, ", ", 2) < 0) {
+                goto error;
+            }
+        }
+        if (PyUnicodeWriter_Format(writer, "name=%R", exc->name) < 0) {
+            goto error;
         }
-        if (PyUnicodeWriter_WriteASCII(writer, "name='", 6) < 0) goto error;
-        if (PyUnicodeWriter_WriteSubstring(writer, exc->name, 0, PyUnicode_GET_LENGTH(exc->name)) < 0) goto error;
-        if (PyUnicodeWriter_WriteASCII(writer, "'", 1) < 0) goto error;
         hasargs = 1;
     }
     if (exc->path) {
         if (hasargs) {
-            if (PyUnicodeWriter_WriteASCII(writer, ", ", 2) < 0) goto error;
+            if (PyUnicodeWriter_WriteASCII(writer, ", ", 2) < 0) {
+                goto error;
+            }
+        }
+        if (PyUnicodeWriter_Format(writer, "path=%R", exc->path) < 0) {
+            goto error;
         }
-        if (PyUnicodeWriter_WriteASCII(writer, "path='", 6) < 0) goto error;
-        if (PyUnicodeWriter_WriteSubstring(writer, exc->path, 0, PyUnicode_GET_LENGTH(exc->path)) < 0) goto error;
-        if (PyUnicodeWriter_WriteASCII(writer, "'", 1) < 0) goto error;
     }
     if (PyUnicodeWriter_WriteASCII(writer, ")", 1) < 0) goto error;
     return PyUnicodeWriter_Finish(writer);