[Mesa-dev] [PATCH] Avoid ASan new-delete-type-mismatch
Eric Engestrom
eric.engestrom at imgtec.com
Fri Jun 10 10:49:19 UTC 2016
On Fri, Jun 10, 2016 at 12:28:38PM +0200, Stephan Bergmann wrote:
> On 06/10/2016 12:09 PM, Eric Engestrom wrote:
> > On Thu, Jun 09, 2016 at 05:13:20PM +0200, Stephan Bergmann wrote:
> >
> > Since you didn't send this patch from your primary email account,
> > you might want to add this line here, otherwise the author will be
> > "stephan.bergmann.secondary at googlemail.com":
> >
> > From: Stephan Bergmann <sbergman at redhat.com>
>
> Add that line where exactly?
At the top, above the commit message. Using `git send-email` on the
patch you originally sent does it automatically (I just double-checked).
>
> (Seriously, this starts to get boring to me. I had a rather trivial
> drive-by patch. I got asked to provide it as exactly "git send-email" (a
> "git format-patch"'ed attachment apparently didn't suit reviewers, for
> whatever reasons),
Inline patches allow for inline comments, which most reviewers prefer.
It also makes the use of git-am much easier, when someone wants to test
your patch, or when a maintainer wants to apply and push your patch.
> so I dutifully set up "git send-email" (which I never use
> otherwise). It didn't work with my primary SMTP server (which I didn't
> bother to investigate into further), so I used another one that happened to
> work. I don't really care with what attribution that change would be
> merged.)
I didn't mean to bother. If you don't care what email address gets
attached to your name, feel free to ignore this :)
Cheers,
Eric
More information about the mesa-dev
mailing list