Hi, Just a remark on the way iterations are going on with this series: I do agree that each version gets better than the previous one, which is good. However, I have the feeling that we're turning a simple and easy to review series into a monster one (reading "v14 .../13" with a non-trivial interdiff is a bit scary for reviewers).
Karthik: I think you could (should?) have splitted the work again. You're integrating other people's idea in the series, and sometimes I think at some point, a better way would have been: "OK, good idea, I'll implement it in on top of this series" (and possibly implement it on top before you resend, to make sure that the series is ready to welcome the new feature). For example, %(contents:lines=X) is good, but could have waited for the next series IMHO. This way, you get a shorter series to converge faster (straightforward interdiff for the last iterations), and then reviewers can focus on the next, short, series. The opposite is Zeno paradox kind of series, where you add something new every time you get close to getting merged, and you actually never reach a stable state ;-). That said, this particular series was a tough one for this, so I'm not even sure my advice would have been applicable ^^. -- Matthieu Moy http://www-verimag.imag.fr/~moy/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
