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

[wluebbe]

This ticket is part of #16073 to simplify review.
In Py2 map() returns a list, while in Py3 map() returns an iterator.

CC: @a-andre

Component: misc

Author: Wilfried Luebbe

Branch/Commit: 21537ab

Reviewer: André Apitzsch

Issue created by migration from https://trac.sagemath.org/ticket/18553

[wluebbe]

comment:1

Another batch of corrections to map().

[wluebbe]

Author: Wilfried Luebbe

[a-andre]

comment:2

Replying to @wluebbe:

Another batch of corrections to map().

Where? There is no branch attached.

[wluebbe]

Branch: u/wluebbe/18553

[wluebbe]

comment:3

Now there is a branch :-/

---

New commits:

98d3263

Trac #18553: The semantic of map() function is changed in Python 3

[wluebbe]

Commit: 98d3263

[a-andre]

comment:5

Could you change in src/sage/geometry/fan.py

-        facets = [frozenset(_) for _ in [c.ambient_ray_indices() for c in self.generating_cones()] ]
+        facets = [frozenset(c.ambient_ray_indices()) for c in self.generating_cones()]

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

21537ab

Implement an improvement as suggested by the reviewer.

[sagetrac-git]

Changed commit from 98d3263 to 21537ab

[wluebbe]

comment:7

Done. Thanx!

[a-andre]

comment:8

LGTM.

[a-andre]

Reviewer: André Apitzsch

[vbraun]

Changed branch from u/wluebbe/18553 to 21537ab