This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Use correct conversion specifications in template string


Hi Jie,

This is OK this time, but in the future, do please include a short
description of what the patch does.  Even if it is obvious.  Even if the
ChangeLog entry says it all.  That way we do not need to download and
open the attachment in order to see what your patch does.

Sorry. I thought everyone uses some email client with text attachment
preview, like Mozilla Thunderbird.

I actually use Thunderbird, but it does not automatically open attachments for me, I still have to click on it in order for it to be opened/downloaded/saved.


Would it be also OK to include such simple patch inline in email?

Certainly. Plus do not let me discourage you from using attachments, especially for larger patches. All I am asking is that if you do use an attachment, please include a short description of it in the main body of your message.


Cheers
  Nick



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]