This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB 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] MI: Add value field to output of -var-create


On Fri, Feb 09, 2007 at 09:07:11AM +1300, Nick Roberts wrote:
> It doesn't display properly in Emacs without a * on each line e.g look at
> this change in Emacs:

Yours doesn't look right in emacs either; spaces instead of tabs
disables the highlighting, and the old date.  I fixed it.

> I kept the *'s but removed the commas.  This style is used by the Emacs
> project, at least.  The convention used there is that if different files
> have different changes then a blank line is left between the ChangeLog entries,
> but if the changes are the same then the entries are contiguous.

I really don't care enough about changelog formatting to argue about
it.  But I will observe one thing: this is GDB, not Emacs, and it's
always more polite to match the style of the file you are changing.
Emacs seems split about 50-50.

-- 
Daniel Jacobowitz
CodeSourcery


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