Apply suggestions from code review

Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
Co-authored-by: Mikhail Efimov <efimov.mikhail@gmail.com>

Author: 3 people

peps/pep-0791.rst

@@ -34,23 +34,23 @@ when explicitly noted otherwise, all return values are floats."  This is no
 longer true:  *None* of the functions listed in the `Number-theoretic
 functions <https://docs.python.org/3.14/library/math.html#number-theoretic-functions>`_
 subsection of the documentation returns a float, but the
-documentation doesn't say so.  In the proposed module a similar sentence "All
-return values are integers." could tell the truth once.  In a similar way we
+documentation doesn't say so.  In the documentation for the proposed ``imath`` module the sentence "All
+return values are integers." would be accurate.  In a similar way we
 can simplify the description of the accepted arguments for functions in both the
 :external+py3.14:mod:`math` and the new module.
 
-Apparently, the :external+py3.14:mod:`math` can't serve as a catch-all place
+Apparently, the :external+py3.14:mod:`math` module can't serve as a catch-all place
 for mathematical functions: we have also the :external+py3.14:mod:`cmath` and
-the :external+py3.14:mod:`statistics`.  Let's do the same for integer-related
-functions.  It would provide shared context, which reduces verbosity in the
+:external+py3.14:mod:`statistics` modules.  Let's do the same for integer-related
+functions.  It provides shared context, which reduces verbosity in the
 documentation and conceptual load.  It also aids discoverability through
-grouping related functions and IDEs suggesting helpful completions.
+grouping related functions and makes IDE suggestions more helpful.
 
 
 Specification
 =============
 
-The PEP proposes moving the following integer-related functions in a new
+The PEP proposes moving the following integer-related functions to a new
 module, called ``imath``:
 
 * :external+py3.14:func:`~math.comb`
@@ -63,11 +63,11 @@ module, called ``imath``:
 Their aliases in :external+py3.14:mod:`math` will be :term:`soft deprecated`.
 
 Module functions will accept integers and objects that implement the
-:external+py3.14:meth:`~object.__index__` method which is used to convert the
+:external+py3.14:meth:`~object.__index__` method, which is used to convert the
 object to an integer number.
 
 Possible extensions for the new module and its scope are discussed in the
-`Open Issues <Open Issues_>`_ section.  New functions are not part of the
+`Open Issues <Open Issues_>`_ section.  New functions are not part of this
 proposal.
 
 
@@ -90,7 +90,7 @@ Open Issues
 Module name
 -----------
 
-Chosen name seems consistent with other domain-specific mathematical module:
+The chosen name seems consistent with one existing domain-specific mathematical module:
 :external+py3.14:mod:`cmath` (for complex numbers).
 
 There is already an ``imath`` project on PyPI, but only with two releases, with
@@ -100,7 +100,7 @@ include Python bindings with the same name.
 
 `Polling shows <https://discuss.python.org/t/91337/35>`_ ``intmath`` as another
 popular name.  The argument made was that the normal mathematical spelling of
-the imaginary unit is ``i``, which makes imath ambiguous.  It also has no conflict
+the imaginary unit is ``i``, which makes ``imath`` ambiguous.  It also has no conflict
 with any PyPI module.  On the other hand, ``intmath`` may be confused with
 interval math or numerical integration.
 
@@ -115,7 +115,7 @@ Unless we can just provide bindings to some well supported mathematical library
 like the GMP, the module scope should be limited.  For example, no efficient primality
 testing and factorization.
 
-There are possible additions, among proposed in the initial discussion thread
+Some possible additions, among those proposed in the initial discussion thread
 (see also issue
 `python/cpython#81313 <https://github.com/python/cpython/issues/81313>`_):
 
@@ -124,10 +124,10 @@ There are possible additions, among proposed in the initial discussion thread
 * ``gcdext()`` --- to solve linear `Diophantine equation <https://en.wikipedia.org/wiki/Diophantine_equation>`_ in two variables (the
   :external+py3.14:class:`int` implementation actually includes an extended
   Euclidean algorithm)
-* ``isqrt_rem()`` --- to return both integer square root and a remainder (if
+* ``isqrt_rem()`` --- to return both integer square root and a remainder (which is non-zero only if
   integer isn't a perfect square)
 * ``ilog()`` --- integer logarithm, :external+py3.14:func:`math.log`
-  has a special handling for integer arguments.  It's unique (with respect to other module
+  has special handling for integer arguments.  It's unique (with respect to other module
   functions) and not documented so far, see issue
   `python/cpython#120950 <https://github.com/python/cpython/issues/120950>`_.
 * ``fibonacci()`` --- `Fibonacci sequence <https://en.wikipedia.org/wiki/Fibonacci_sequence>`_.
@@ -146,8 +146,8 @@ would be confusing to give it the same name (under a different module).
 Acknowledgements
 ================
 
-Thanks to Tim Peters for reviving the idea of the :external+py3.14:mod:`math`
-splitting.  Thanks to Neil Girdhar for substantial improvements of
+Thanks to Tim Peters for reviving the idea of splitting the :external+py3.14:mod:`math`
+module.  Thanks to Neil Girdhar for substantial improvements of
 the initial draft.