Address feedback from review.

Author: ZeroIntensity

peps/pep-0788.rst

@@ -520,10 +520,20 @@ With this PEP, you would implement it like this:
             return -1;
         }
 
-        char *to_write = do_some_text_mutation(text);
+        const char *to_write = PyUnicode_AsUTF8(text);
+        if (to_write == NULL) {
+            // Since the exception may be destroyed upon calling PyThreadState_Release(),
+            // print out the exception ourself.
+            PyErr_Print();
+            PyThreadState_Release(thread_view);
+            PyInterpreterLock_Release(lock);
+            return -1;
+        }
         int res = PyFile_WriteString(to_write, file);
         free(to_write);
-        PyErr_Print();
+        if (res < 0) {
+            PyErr_Print();
+        }
 
         PyThreadState_Release(thread_view);
         PyInterpreterLock_Release(lock);
@@ -703,8 +713,8 @@ Example: An Asynchronous Callback
     static int
     async_callback(void *arg)
     {
-        ThreadData *data = (ThreadData *)arg;
-        PyInterpreterView view = data->view;
+        ThreadData *tdata = (ThreadData *)arg;
+        PyInterpreterView view = tdata->view;
         PyInterpreterLock lock = PyInterpreterLock_AcquireView(view);
         if (lock == 0) {
             fputs("Python has shut down!\n", stderr);
@@ -721,6 +731,8 @@ Example: An Asynchronous Callback
         }
         PyThreadState_Release(thread_view);
         PyInterpreterLock_Release(lock);
+        PyInterpreterView_Close(view);
+        PyMem_RawFree(tdata);
         return 0;
     }