#16409: 2to3 refactoring tool has warnings for some code
---------------------------+----------------------------
   Reporter:  wluebbe      |            Owner:
       Type:  enhancement  |           Status:  new
   Priority:  minor        |        Milestone:  sage-6.3
  Component:  build        |         Keywords:  python3
  Merged in:               |          Authors:
  Reviewers:               |  Report Upstream:  N/A
Work issues:               |           Branch:
     Commit:               |     Dependencies:
   Stopgaps:               |
---------------------------+----------------------------
 While running the Python 2to3 on the Sage code base some warning are
 emitted:
 * fixer= absolute_import
 {{{
 ### In file sage/misc/sage_timeit.py ###
 Line 199: absolute and local imports together
 }}}

 * fixer= map
 {{{
 ### In file sage/geometry/polyhedron/constructor.py ###
 Line 364: You should use a for loop here
 ### In file sage/geometry/polyhedron/ppl_lattice_polytope.py ###
 Line 1078: You should use a for loop here
 }}}

 * fixer= next
 {{{
 ### In file sage/combinat/integer_list.py ###
 Line 322: Calls to builtin next() possibly shadowed by global binding
 }}}

 * fixer= operator
 {{{
 ### In file sage/combinat/sf/sfa.py ###
 Line 3520: You should use 'operator.mul(lcm(pair), times=gcd(pair))' here.
 ### In file sage/combinat/species/generating_series.py ###
 Line 724: You should use 'operator.mul(lcm(pair), times=gcd(pair))' here.
 }}}

--
Ticket URL: <http://trac.sagemath.org/ticket/16409>
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.

Reply via email to