Opened 2 years ago

Closed 2 years ago

#24185 closed enhancement (wontfix)

py3: wrapping map with list in typeset

Reported by: chapoton Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: python3 Keywords:
Cc: Merged in:
Authors: Frédéric Chapoton Reviewers:
Report Upstream: N/A Work issues:
Branch: u/chapoton/24185 Commit: c479e313ef9b433eb716032900833d5259af92f8
Dependencies: Stopgaps:


Change History (4)

comment:1 Changed 2 years ago by chapoton

  • Branch set to u/chapoton/24185
  • Cc tscrim jdemeyer jhpalmieri added
  • Commit set to c479e313ef9b433eb716032900833d5259af92f8
  • Status changed from new to needs_review

New commits:

c479e31py3: wrapping map with list in typeset/

comment:2 Changed 2 years ago by chapoton

  • Cc tscrim jdemeyer jhpalmieri removed
  • Milestone changed from sage-8.1 to sage-duplicate/invalid/wontfix
  • Status changed from needs_review to positive_review

does not this what it was supposed to.. let us close that

comment:3 Changed 2 years ago by tscrim

I think this change should not needed, at least the concatenation function should handle any iterable as input.

comment:4 Changed 2 years ago by embray

  • Resolution set to wontfix
  • Status changed from positive_review to closed

Plus in general, while there are many bugs on Python 3 where the return value of map() is assumed to be a list, I think a better approach to those cases is often to convert the map() call to a list comprehension. That way you aren't needlessly copying the list on Python 2.

Note: See TracTickets for help on using tickets.