On Sun, Aug 2, 2015 at 1:01 AM, Eric Sunshine <sunsh...@sunshineco.com> wrote: > Don't worry too much about it. Consider it something to keep in mind for > future patches. I reviewed the change and it seemed okay. I mentioned it > because one of the goals of patch submission, in addition to making an > actual change, is to ease the review process. If Junio is okay with > accepting it as is, then it's probably not worth spending more time > trying to refine it. > > Having said that, I came up with the following for those two paragraphs, > which gives a much less noisy diff and doesn't look too jagged: >
Yea, I actually have re-worked it as well to something more suitable. I'm re-sending anyways as I fixed some of the other style issues and the one bug about not returning 0 on good read of the notes.merge value. Regards Jake -- 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
