Message234771
| Author | Joshua.Landau |
|---|---|
| Recipients | Jeff.Kaufman, Joshua.Landau, NeilGirdhar, SpaghettiToastBook, andybuckley, belopolsky, berker.peksag, eric.araujo, eric.snow, ezio.melotti, georg.brandl, gvanrossum, ncoghlan, paul.moore, pconnell, r.david.murray, terry.reedy, twouters, zbysz |
| Date | 2015-01-26.19:18:18 |
| SpamBayes Score | -1.0 |
| Marked as misclassified | Yes |
| Message-id | <1422299912.93.0.468871144722.issue2292@psf.upfronthosting.co.za> |
| In-reply-to |
| Content | |
|---|---|
Quick-fix for Guido's bug attached. I'm not familiar with this part of the code, yet, so take this tentatively. I just changed
while (containers > 1) {
to
while (containers) {
---
@Guido
My comments were assuming `f(**x for x in y)` meant `f({**x for x in y})`.
I see your reasoning, but I don't like how your version has
(x for y in z for x in y) == (*y for y in z)
f(x for y in z for x in y) != f(*y for y in z)
This seems like a tripping point. I've never wanted to unpack a 2D iterable into an argument list, so personally I'm not convinced by the value-add either. |
|
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2015-01-26 19:18:33 | Joshua.Landau | set | recipients: + Joshua.Landau, gvanrossum, twouters, georg.brandl, terry.reedy, paul.moore, ncoghlan, belopolsky, ezio.melotti, eric.araujo, andybuckley, r.david.murray, zbysz, eric.snow, berker.peksag, pconnell, NeilGirdhar, Jeff.Kaufman, SpaghettiToastBook |
| 2015-01-26 19:18:32 | Joshua.Landau | set | messageid: <1422299912.93.0.468871144722.issue2292@psf.upfronthosting.co.za> |
| 2015-01-26 19:18:32 | Joshua.Landau | link | issue2292 messages |
| 2015-01-26 19:18:32 | Joshua.Landau | create | |