[PATCH 1 of 4 py3] maplist: replace instances of map() with pycompat.maplist() [1 of 4]

Augie Fackler raf at durin42.com
Fri Mar 24 14:07:08 EDT 2017


On Sat, Mar 25, 2017 at 12:49:05AM +0900, Yuya Nishihara wrote:
> On Fri, 24 Mar 2017 21:00:24 +0530, Pulkit Goyal wrote:
> > This series is a part of fixing things which we already know will lead to
> > exception or wrong result. So instead of waiting for those exceptions to
> > pop up, it fixes them. It fixes all the occurrences in mercurial/ (couple
> > of occurrences are left where using maplist was not required).
>
> Not all map() are needed to be converted to maplist(). If we don't want to
> sort out them, we can simply redirect map to maplist by importer.

Yeah, I'd rather we trip over cases where we used map
not-as-an-iterator and fix them incrementally rather than just
touching them all wholesale.

> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


More information about the Mercurial-devel mailing list