Message260987
| Author |
eric.fahlgren |
| Recipients |
Barun Parruck, eric.fahlgren, llllllllll, ncoghlan, yselivanov |
| Date |
2016年02月28日.21:02:22 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1456693343.08.0.978322543002.issue26448@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
Two things:
1) Verified this has always been a problem and still is in the development branch, so I added 2.7 and 3.6 to the versions list.
2) Couldn't tolerate the duplicate code handling the extended args operator, so in the interests of DRY, I moved the code scanner to a generator function (see attached dis_with_code_scanner.diff).
The patch is definitely not required to fix this bug, but it does isolate the original problem area to just one piece of code.
Could we get a get a quick review from a core dev saying either "go with Barun's less invasive preliminarypatch.diff" or "go with Eric's greater-churn dis_with_code_scanner.diff patch?"
(We still need to flesh out the testing a bit, so don't call us, we'll call you. :) ) |
|