Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#18474 closed enhancement (fixed)

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

Reported by: aapitzsch Owned by:
Priority: major Milestone: sage-6.8
Component: misc Keywords: python3
Cc: jdemeyer Merged in:
Authors: André Apitzsch Reviewers: Wilfried Luebbe
Report Upstream: N/A Work issues:
Branch: 237ac81 (Commits) Commit:
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 aapitzsch

  • Cc jdemeyer added
  • Status changed from new to needs_review

comment:2 Changed 5 years ago by wluebbe

  • Reviewers set to Wilfried Lübbe
  • Status changed from needs_review to positive_review

All test passed and the changes look good to me.

There are still some overlooked maps in the changed files. But I will create a followup ticket/patch to address those (so that the effort to review the current patch is not wasted).

comment:3 Changed 5 years ago by vbraun

  • Branch changed from u/aapitzsch/map_p3 to 237ac81bd66a22f6e3814408d2153a9ecf2cf66b
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:4 Changed 5 years ago by jdemeyer

  • Commit 237ac81bd66a22f6e3814408d2153a9ecf2cf66b deleted
  • Reviewers changed from Wilfried Lübbe to Wilfried Luebbe
Note: See TracTickets for help on using tickets.