On Sun, Jun 9, 2013 at 6:22 PM, Célestin Matte <celestin.matte@xxxxxxxxxx> wrote: > Changes with v2: > - Remove patch [02/22] about using the Readonly module > - Split commit [07/22] into 5 different ones This was easier to review after being split. Thanks. > - Split commit [14/22] into 2 different ones > - Patch [17/22] was *not* split: tell me if it is necessary [now patch 22/28] You, Matthieu, and Junio should decide, but I again found it time-consuming and onerous to review with all the changes mashed together. > - Remove wrong change in patch [22/22] > > Patch about fixing a bug in a regexp (with has been renames "Make a regexp > clearer"), which had been sent as a separate patch, is reintegrated into this > series of patches. Overall, this series was more pleasant and easy to read than previous versions. With the exception of minor comments, questions, and the whitespace noise in 22/28, I did not spot any issues worth mentioning in the remainder of the series. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html