Hi Krisztián,

On Wed, Sep 25, 2019 at 08:09:26PM +0000, Krisztián Kovács (kkovacs) wrote:
> Thanks for your comments. Hopefully both are fixed. I've attached the
> updated patch.

Thank you, that's perfect now, and merged.

> (Is attaching the proper way to follow up with fixes?)

Both attaching and inlining are fine to me, as long as the attachment
is tagged text/plain so that the mailer will inline it in responses
in case a review is needed. I admit having a slight preference for
attachments over inline for the sole reason that I don't have to invent
a file name to save the file if I apply it remotely, but that's extremely
minor as you can guess. Anyway in your case it was perfect, so please do
not change anything :-)


Reply via email to