Opened 5 years ago

Closed 5 years ago

#18531 closed enhancement (fixed)

Python 3: The semantic of map() function is changed (addendum 1)

Reported by: wluebbe Owned by:
Priority: major Milestone: sage-6.8
Component: misc Keywords: python3
Cc: aapitzsch Merged in:
Authors: Wilfried Luebbe Reviewers: André Apitzsch
Report Upstream: N/A Work issues:
Branch: 93673ca (Commits) Commit: 93673cae09958c4024a10b37a466526c78c37ecb
Dependencies: Stopgaps:

Description

This ticket is part of #16073 to simplify review.

In Py2 map() returns a list, while in Py3 map() returns an iterator.

Change History (4)

comment:1 Changed 5 years ago by wluebbe

  • Authors set to Wilfried Luebbe
  • Branch set to u/wluebbe/18531
  • Commit set to 150f5a2d3bccdd096fe200c47e3b6cc9da6ee428
  • Status changed from new to needs_review
  • Summary changed from Python 3: The semantic of map() function is changed (appendix) to Python 3: The semantic of map() function is changed (addendum 1)
  • Type changed from PLEASE CHANGE to enhancement

A new batch of corrections to map().


New commits:

150f5a2Trac #18531: The semantic of map() function is changed in Python 3

comment:2 Changed 5 years ago by aapitzsch

  • Branch changed from u/wluebbe/18531 to public/18531
  • Commit changed from 150f5a2d3bccdd096fe200c47e3b6cc9da6ee428 to 93673cae09958c4024a10b37a466526c78c37ecb
  • Reviewers set to André Apitzsch

I removed a trailing white space and replaced _ by a nicer looking i in some examples. And letters is already a list, so no need to convert it to one.

If you are happy with my changes, set the ticket to positive review.

comment:3 Changed 5 years ago by wluebbe

  • Status changed from needs_review to positive_review

Your changes are fine and the long tests are still OK.

comment:4 Changed 5 years ago by vbraun

  • Branch changed from public/18531 to 93673cae09958c4024a10b37a466526c78c37ecb
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.