More thoughts.

0. It's good that you are refreshing this part of the documentation,
if only to update references to Pyrex to references to Cython!

1. Maybe it would be worth pointing to the subtle ways in which these
functions differ in names and in nature:

    There are three relevant functions, with related names
    (note the add vs iadd, and the single vs double underscores),
    one of which has both a Python and a Cython version (note
    the def vs cpdef).

and, in the paragraph about _add_, to insist once again
on the presence of the def and cpdef versions.

2. In the sentence

    For speed, there are also inplace version of the arithmetic commands.

we want "inplace versions" (plural).

3. In the sentence

    This is the function you should override to inplace implement addition
    in a Python subclass of RingElement.

I would change "to inplace implement addition" to "to implement
inplace addition".

-- 
You received this message because you are subscribed to the Google Groups 
"sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sage-devel+unsubscr...@googlegroups.com.
To post to this group, send email to sage-devel@googlegroups.com.
Visit this group at http://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.

Reply via email to