#12415: Update doctesting framework
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
Reporter: robertwb
| Owner: mvngu
Type: enhancement
| Status: needs_work
Priority: major
| Milestone: sage-5.7
Component: doctest
| Resolution:
Keywords:
| Work issues:
Report Upstream: N/A
| Reviewers: Jeroen Demeyer
Authors: David Roe, Robert Bradshaw
| Merged in:
Dependencies: #13147,#13146, #13145, #12723, #12392, #12393, #12395,
#12396, #12397, #12381, #12382, #12383, #12384, #11871, #13195, #13121, #13748,
#13899, #12719, #5155 | Stopgaps:
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
Comment (by jdemeyer):
There is a lot of
{{{
if output_semaphore is not None:
output_semaphore.acquire()
}}}
in the code which could be simplified assuming that `output_semaphore` is
never `None`. Could we ensure that?
In fact, the whole block
{{{
try:
if output_semaphore is not None:
output_semaphore.acquire()
do_something()
finally:
if output_semaphore is not None:
output_semaphore.release()
}}}
could be simplified to
{{{
with output_semaphore:
do_something()
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12415#comment:162>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
