Entry for Modula-2 changes in NEWS?

Entry for Modula-2 changes in NEWS?

[Joel Brobecker]

Hello,

Should we add an entry for the Modula-2 changes. The subject says
that support for Modula-2 has been enhanced, so it seems worth
detailing this a bit more in the NEWS file. Opinions?

-- 
Joel

Re: Entry for Modula-2 changes in NEWS?

[Gaius Mulley]

Joel Brobecker <brobecker@adacore.com> writes:

> Hello,
> 
> Should we add an entry for the Modula-2 changes. The subject says
> that support for Modula-2 has been enhanced, so it seems worth
> detailing this a bit more in the NEWS file. Opinions?

Hi Joel,

feel free to use these words (or modify them in any way) if you
decide that detail is warranted..

regards,
Gaius


diff -r1.187 NEWS
57a58,66
> * Improved Modula-2 language support
>
> GDB can now print most types in the Modula-2 syntax.  This includes:
> basic types, set types, record types, enumerated types, range types,
> pointer types and ARRAY types.  Procedure var parameters are
> correctly
> printed and hexadecimal addresses and character constants are also
> written in the Modula-2 syntax.  Best results can be obtained by
> using GNU Modula-2 together with the -gdwarf-2 command line option.
>

Re: Entry for Modula-2 changes in NEWS?

[Nick Roberts]

 > I am happy to submit these changes for approval, but I think it would
 > make more sense if you did it. What do you think? Once approved, I can
 > port the change to the branch as well, if you prefer. It's not
 > difficult, but you need to check out the branch, which takes a bit of
 > time. I already have done it on my side.
 
Or presumably copy the working directory and update to the branch:

cp -r src src_6_5
cd src_6_5
cvs up -r gdb_6_5-branch

which should be a lot quicker.


-- 
Nick                                           http://www.inet.net.nz/~nickrob

Re: Entry for Modula-2 changes in NEWS?

[Daniel Jacobowitz]

On Wed, May 17, 2006 at 08:42:55AM -0700, Joel Brobecker wrote:
> > > using GNU Modula-2 together with the -gdwarf-2 command line option.
>                                          ^^^^^^^^^ -gdwarf2 or -gdwarf-2
>                                          ('-' or no '-'?)

-gdwarf-2, with dash.  -gdwarf2 at one point would have meant "DWARF-1,
verbosity level 2".

-- 
Daniel Jacobowitz
CodeSourcery

Re: Entry for Modula-2 changes in NEWS?

[Gaius Mulley]

Daniel Jacobowitz <drow@false.org> writes:

> On Wed, May 17, 2006 at 08:42:55AM -0700, Joel Brobecker wrote:
> > > > using GNU Modula-2 together with the -gdwarf-2 command line option.
> >                                          ^^^^^^^^^ -gdwarf2 or -gdwarf-2
> >                                          ('-' or no '-'?)
> 
> -gdwarf-2, with dash.  -gdwarf2 at one point would have meant "DWARF-1,
> verbosity level 2".

Hi,

yes "-gdwarf-2" with the dash, the option which generates dwarf2
debugging information using "gcc-3.3.6/gcc/dwarf2out.c" say..

regards,
Gaius

Re: Entry for Modula-2 changes in NEWS?

[Gaius Mulley]

Nick Roberts <nickrob@snap.net.nz> writes:

>  > I am happy to submit these changes for approval, but I think it would
>  > make more sense if you did it. What do you think? Once approved, I can
>  > port the change to the branch as well, if you prefer. It's not
>  > difficult, but you need to check out the branch, which takes a bit of
>  > time. I already have done it on my side.
>  
> Or presumably copy the working directory and update to the branch:
> 
> cp -r src src_6_5
> cd src_6_5
> cvs up -r gdb_6_5-branch
> 
> which should be a lot quicker.

Hi,

thanks for that trick..

regards,
Gaius

Re: Entry for Modula-2 changes in NEWS?

[Gaius Mulley]

Joel Brobecker <brobecker@adacore.com> writes:

> Hi Gaius,
> 
> > feel free to use these words (or modify them in any way) if you
> > decide that detail is warranted..
> 
> I am happy to submit these changes for approval, but I think it would
> make more sense if you did it. What do you think? Once approved, I can
> port the change to the branch as well, if you prefer. It's not
> difficult, but you need to check out the branch, which takes a bit of
> time. I already have done it on my side.

all done now,

> 
> > > * Improved Modula-2 language support
> > >
> > > GDB can now print most types in the Modula-2 syntax.  This includes:
> > > basic types, set types, record types, enumerated types, range types,
> > > pointer types and ARRAY types.  Procedure var parameters are
> > > correctly
> > > printed and hexadecimal addresses and character constants are also
> > > written in the Modula-2 syntax.  Best results can be obtained by
> > > using GNU Modula-2 together with the -gdwarf-2 command line option.
>                                          ^^^^^^^^^ -gdwarf2 or -gdwarf-2
>                                          ('-' or no '-'?)


yes with the '-2'

regards,
Gaius

Re: Entry for Modula-2 changes in NEWS?

[Daniel Jacobowitz]

On Fri, May 19, 2006 at 03:16:08AM +0100, Gaius Mulley wrote:
> Joel Brobecker <brobecker@adacore.com> writes:
> 
> > Hi Gaius,
> > 
> > > feel free to use these words (or modify them in any way) if you
> > > decide that detail is warranted..
> > 
> > I am happy to submit these changes for approval, but I think it would
> > make more sense if you did it. What do you think? Once approved, I can
> > port the change to the branch as well, if you prefer. It's not
> > difficult, but you need to check out the branch, which takes a bit of
> > time. I already have done it on my side.
> 
> all done now,

I see you've modified NEWS on the branch.  Please, three notes:

- Patches need to be sent to gdb-patches@.

- Changes shouldn't go onto a branch that haven't already gone onto
  HEAD.

- And it's usually polite to post patches before checking them in;
  for instance, Eli usually reviews documentation patches quickly
  (and NEWS is part of the documentation now).

-- 
Daniel Jacobowitz
CodeSourcery

[patch] update for Modula-2 changes in NEWS

[Gaius Mulley]

Joel Brobecker <brobecker@adacore.com> writes:

> Hi Gaius,
> 
> > feel free to use these words (or modify them in any way) if you
> > decide that detail is warranted..
> 
> I am happy to submit these changes for approval, but I think it
> would make more sense if you did it. What do you think? Once
> approved, I can port the change to the branch as well, if you
> prefer. It's not difficult, but you need to check out the branch,
> which takes a bit of time. I already have done it on my side.

Hi Joel,

I've applied the diff to the branch 6_5 and then realised I jumped the
gun so have retracted the patch and I'm submitting it here for
approval:

diff -r1.188 NEWS
61a62,70
> * Improved Modula-2 language support
> 
> GDB can now print most types in the Modula-2 syntax.  This includes:
> basic types, set types, record types, enumerated types, range types,
> pointer types and ARRAY types.  Procedure var parameters are correctly
> printed and hexadecimal addresses and character constants are also
> written in the Modula-2 syntax.  Best results can be obtained by using
> GNU Modula-2 together with the -gdwarf-2 command line option.
> 

apologies,
Gaius

Re: Entry for Modula-2 changes in NEWS?

[Gaius Mulley]

Daniel Jacobowitz <drow@false.org> writes:

> 
> I see you've modified NEWS on the branch.  Please, three notes:
> 
> - Patches need to be sent to gdb-patches@.
> 
> - Changes shouldn't go onto a branch that haven't already gone onto
>   HEAD.
> 
> - And it's usually polite to post patches before checking them in;
>   for instance, Eli usually reviews documentation patches quickly
>   (and NEWS is part of the documentation now).
> 

Hi Daniel,

very sorry - many apologies,
Gaius

Re: [patch] update for Modula-2 changes in NEWS

[Joel Brobecker]

> I've applied the diff to the branch 6_5 and then realised I jumped the
> gun so have retracted the patch and I'm submitting it here for
> approval:

That's great. Thanks! Just a quick note, before Eli takes a look:
You forgot to include a ChangeLog entry with your patch.

> diff -r1.188 NEWS
> 61a62,70
> > * Improved Modula-2 language support
> > 
> > GDB can now print most types in the Modula-2 syntax.  This includes:
> > basic types, set types, record types, enumerated types, range types,
> > pointer types and ARRAY types.  Procedure var parameters are correctly
> > printed and hexadecimal addresses and character constants are also
> > written in the Modula-2 syntax.  Best results can be obtained by using
> > GNU Modula-2 together with the -gdwarf-2 command line option.

-- 
Joel