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@mercurial-scm.org > https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel