Re: [PATCH v4 11/16] am -3: use merge_recursive() directly again

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes:

> If you want, I can break out the subsequent patches into a separate
> series.

I do not think that would help anybody, as we'll have to review them
anyway.  If some of the the later ones were "oops, this earlier step
did an incomplete job and here is a fix-up", then squashing them
into the step where such a mistake happens may reduce the review
load, but I suspect that is not the case (iow, they are enhancements
and the earlier ones can stand on their own).

> I may have missed something as stupid as an unclosed file handle, after
> all.

Yes, reviewing the same change over and over, especially if the
change is your own, tends to have diminishing rate of bug yields,
which is why they need to be reviewed by fresh set of eyes.



--
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



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]