Re: [PATCH v3 00/11] unify appending of sob
- From
Brandon Casey <drafnel@gmail.com>
- Date
- Jan 31, 2013, 18:45 UTC
- Message-ID
- <CA+sFfMe4NYWKQBM0N0isBfF+RY2AemKugKT+RyA98boO-p1rYQ@mail.gmail.com>
- In-Reply-To
- <7vhalylggd.fsf@alter.siamese.dyndns.org>
On Wed, Jan 30, 2013 at 9:37 AM, Junio C Hamano <gitster@pobox.com> wrote:
Show 27 quoted lines
> Junio C Hamano <gitster@pobox.com> writes: > >> Jonathan Nieder <jrnieder@gmail.com> writes: >> >>> Brandon Casey wrote: >>> >>>> Round 3. >>> >>> Thanks for a pleasant read. My only remaining observations are >>> cosmetic, except for a portability question in Duy's test script, a >>> small behavior change when the commit message ends with an >>> RFC2822-style header with no trailing newline and the possibility of >>> tightening the pattern in sequencer.c to match the strictness of >>> format-patch (which could easily wait for a later patch). >> >> Thanks for a quick review. I agree that this series is getting very >> close with your help. > > Unless Brandon and/or Jonathan wants to have another chance to > excise warts from the recorded history by rerolling the entire > series one more time, I think what we have queued is in a good > enough shape to merge to 'next' and any further improvement and fix > can be done incrementally. > > OK? Or "stop, I want to reroll"? > > I'll wait for a day or two.
Let's hold off so I can do another round. I worked on this last night and was able to simplify some things nicely. I'll try to finish up tonight and resubmit.
-Brandon