Karthik Nayak <karthik.188@xxxxxxxxx> writes: > On 05/29/2015 01:56 AM, Matthieu Moy wrote: >> Karthik Nayak <karthik.188@xxxxxxxxx> writes: >> >>> Rename 'refinfo' to 'ref_array_item' and intoduce 'ref_filter_cbdata' >> >> The fact that you need to use "and" to describe your changes is a hint >> that you can split better. >> > > But the patch alone wouldn't make much sense here, as the whole idea > is the introduction of the new structures and renaming 'refinfo' to > ref_array_item' was more of a byproduct to go along with the new > structures introduced. The point of separating them is that the rename implies a relatively long patch (you have 17 occurences of 'refinfo' in the deletion part of your patch), but it is straightforward to review (apply, run "git diff --color-words" and press space a few times). But it is no longer so simple once you mix it with anything else. -- 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html