#18626: fix gap_packages (Guava) optional tests
-------------------------------------+-------------------------------------
Reporter: dimpase | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.8
Component: packages: | Resolution:
optional | Merged in:
Keywords: | Reviewers:
Authors: Dima Pasechnik | Work issues:
Report Upstream: Fixed upstream, | Commit:
in a later stable release. | a26ecb319bf7b5b9239e3d925407c48b1ace7d86
Branch: u/dimpase/18626 | Stopgaps:
Dependencies: #18689 |
-------------------------------------+-------------------------------------
Comment (by dimpase):
Replying to [comment:22 ncohen]:
> > IMHO, `continue` is even less readable than `goto`...
>
> With a 'continue' you do not have to change the indentation of 10 lines
of code only because you want to make an assumption.
well, but it results in unreadable crap. Given that this loop already has
a number of `break`s,
in particular.
>
> And I will start minding 'continue' the day Python will be compiled. A
language that stores classes methods in a *dictionary* does not deserve
this kind of respect.
Language, perhaps. But the mere mortals who need to read the code?
--
Ticket URL: <http://trac.sagemath.org/ticket/18626#comment:23>
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.
For more options, visit https://groups.google.com/d/optout.
