Bring matrix/matrix0.pyx to 100% coverage

[hughrthomas]

Improve the doctests for matrix/matrix0.pyx.

---

Apply attachment: trac_12585_matrix0_doc-ht.patch and attachment: trac_12585-review.patch.

CC: @hughrthomas

Component: doctest coverage

Author: Hugh Thomas

Reviewer: David Loeffler, Karl-Dieter Crisman

Merged: sage-5.0.beta8

Issue created by migration from https://trac.sagemath.org/ticket/12585

[hughrthomas]

comment:1

I think this covers it. sage -coverage still complains about one thing, but I think it's confused (the thing it's found is something defined by cdef but which is not a method).

[hughrthomas]

comment:2

Drat. New patch coming

[hughrthomas]

Attachment: trac_12585_matrix0_doc-ht.patch.gz

[hughrthomas]

comment:3

This one applies properly to 5.0.beta5.

[loefflerd]

comment:4

The reason sage -coverage was complaining is because of the comment placed before the docstring. I.e. it's not clever enough to parse

def some_function(args):
    # comment
    """ docstring """
    function_body

I'm about to upload a tiny reviewer patch which moves the comment in _lmul_ to after the docstring, and thus gets coverage up to 100%. I've also tagged a few doctests as # indirect doctest to stop it complaining about the function name not appearing. Hugh: I'm happy with your patch, so if you're happy with my patch please set this to "positive review".