Remove value judgements

Author: RadicalZephyr

Doc/library/pty.rst

@@ -48,15 +48,19 @@ The :mod:`pty` module defines the following functions:
    will return.
 
    The functions *master_read* and *stdin_read* are passed a file descriptor
-   which they should read from, and they should always return a byte
-   string. Returning an empty byte string from either callback is interpreted as
-   an end-of-file (EOF) condition, and that callback will not be called after
-   that. This is not recommended. Instead, the graceful way to force the *spawn*
-   to return before the child process exits is to raise *OsError*.
-
-   In general, manually signaling EOF without receiving it from the underlying
-   file descriptor from one or both of the read callbacks will not behave
-   well. If *stdin_read* signals EOF the controlling terminal can no longer
+   which they should read from, and they should always return a byte string. In
+   order to force spawn to return before the child process exits an *OsError*
+   should be thrown.
+
+   The default implementation for both functions will read and return up to 1024
+   bytes each time the function is called. *Master_read* is passed the
+   pseudoterminal’s master file descriptor to read output from the child
+   process, and *stdin_read* is passed file descriptor 0, to read from the
+   parent process's standard input.
+
+   Returning an empty byte string from either callback is interpreted as an
+   end-of-file (EOF) condition, and that callback will not be called after
+   that. If *stdin_read* signals EOF the controlling terminal can no longer
    communicate with the parent process OR the child process. Unless the child
    process will quit without any input, *spawn* will then loop forever. If
    *master_read* signals EOF the same behavior results (on linux at least).
@@ -65,11 +69,6 @@ The :mod:`pty` module defines the following functions:
    *select* throws an error on your platform when passed three empty lists. This
    is a bug, documented in `issue 26228 <https://bugs.python.org/issue26228>`_.
 
-   The default implementation for both functions will read and return up to 1024
-   bytes each time the function is called. *Master_read* is passed the
-   pseudoterminal’s master file descriptor to read output from the child
-   process, and *stdin_read* is passed file descriptor 0, to read from the
-   parent process's standard input.
 
    .. versionchanged:: 3.4
       :func:`spawn` now returns the status value from :func:`os.waitpid`