Jeff King writes:
> I wonder if it would be sane to remove or quote NULs when attaching the
> buffer to commit->buffer. That would _break_ signatures, but that is a
> good thing. I do not think there is a reason to have NULs in your commit
> message unless you are doing something malicious (or us
Junio C Hamano writes:
> Jeff King writes:
>
>> Subject: [PATCH] reuse commit->buffer when parsing signatures
>> ...
>> Signed-off-by: Jeff King
>
> Hmph, unfortunately this seems to break t7510.
And I think without re-reading the patch I know what is wrong. The
length of the object and strle
On Fri, May 30, 2014 at 01:44:32PM -0700, Junio C Hamano wrote:
> Jeff King writes:
>
> > Subject: [PATCH] reuse commit->buffer when parsing signatures
> > ...
> > Signed-off-by: Jeff King
>
> Hmph, unfortunately this seems to break t7510.
Urgh, sorry for not testing more thoroughly.
I imagi
Jeff King writes:
> Subject: [PATCH] reuse commit->buffer when parsing signatures
> ...
> Signed-off-by: Jeff King
Hmph, unfortunately this seems to break t7510.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordom
On Fri, May 30, 2014 at 02:34:41PM -0400, Jeff King wrote:
> On Fri, May 30, 2014 at 10:35:14AM -0700, Junio C Hamano wrote:
>
> > > Do you want me to roll it up with a real commit message?
> >
> > Yes. I think the change is sensible.
>
> Here it is. [...]
By the way, I rather derailed Linus'
On Fri, May 30, 2014 at 10:35:14AM -0700, Junio C Hamano wrote:
> > Do you want me to roll it up with a real commit message?
>
> Yes. I think the change is sensible.
Here it is. We may want to make these helper functions available to
other callers so they can use the same trick, but I do not kn
Jeff King writes:
> On Fri, May 30, 2014 at 09:55:14AM -0700, Junio C Hamano wrote:
>
> I don't think we need to worry about commit->buffer being mucked with.
> It is always either NULL, or points to the original object contents.
> Encoded log messages are always placed in a separate buffer (and
On Fri, May 30, 2014 at 09:55:14AM -0700, Junio C Hamano wrote:
> Jeff King writes:
>
> > On Thu, May 29, 2014 at 09:54:10PM -0700, Linus Torvalds wrote:
> >
> >> That said, part of it is just that show-signature is so suboptimal
> >> performance-wise, re-parsing the commit buffer for each commi
Jeff King writes:
> On Thu, May 29, 2014 at 09:54:10PM -0700, Linus Torvalds wrote:
>
>> That said, part of it is just that show-signature is so suboptimal
>> performance-wise, re-parsing the commit buffer for each commit when
>> "show_signature" is set. That's just crazy, we've already parsed th
On Thu, May 29, 2014 at 09:54:10PM -0700, Linus Torvalds wrote:
> That said, part of it is just that show-signature is so suboptimal
> performance-wise, re-parsing the commit buffer for each commit when
> "show_signature" is set. That's just crazy, we've already parsed the
> commit text, we alread
On Thu, May 29, 2014 at 6:58 PM, Jeff King wrote:
>
> I will spare you the usual lecture on having these lines in the message
> body. ;)
We do it for the kernel because they often get lost otherwise.
Particularly the date/author.
git doesn't tend to have the same kind of deep email forwarding
mo
On Thu, May 29, 2014 at 03:31:58PM -0700, Linus Torvalds wrote:
> From: Linus Torvalds
> Date: Thu, 29 May 2014 15:19:40 -0700
> Subject: [RFC PATCH] git log: support "auto" decorations
I will spare you the usual lecture on having these lines in the message
body. ;)
>
From: Linus Torvalds
Date: Thu, 29 May 2014 15:19:40 -0700
Subject: [RFC PATCH] git log: support "auto" decorations
This works kind of like "--color=auto" - add decorations for interactive
use, but do not change defaults when scripting or when piping the output
to anything
13 matches
Mail list logo