Re: [PATCH 7/7] PR gdb/15224 should "set history save on" by default

Eli Zaretskii
2013-05-08T16:51:28Z

> Date: Wed, 8 May 2013 13:24:41 +0500
> From: mbilal <mbilal@codesourcery.com>
>
> On 05/06/2013 02:50 PM, mbilal wrote:
> > Here is doc and news entry.
> >
> > Index: doc/gdb.texinfo
> > ===================================================================
> > RCS file: /cvs/src/src/gdb/doc/gdb.texinfo,v
> > retrieving revision 1.1084
> > diff -u -p -r1.1084 gdb.texinfo
> > --- doc/gdb.texinfo 29 Apr 2013 17:32:43 -0000 1.1084
> > +++ doc/gdb.texinfo 6 May 2013 09:28:27 -0000
> > @@ -21286,7 +21286,7 @@ is not set.
> > @item set history save
> > @itemx set history save on
> > Record command history in a file, whose name may be specified with the
> > -@code{set history filename} command. By default, this option is
> > disabled.
> > +@code{set history filename} command. By default, this option is
> > enabled.
> >
> > @item set history save off
> > Stop recording command history in a file.
> > Index: doc/ChangeLog
> > ===================================================================
> > RCS file: /cvs/src/src/gdb/doc/ChangeLog,v
> > retrieving revision 1.1452
> > diff -u -p -r1.1452 ChangeLog
> > --- doc/ChangeLog 29 Apr 2013 17:32:43 -0000 1.1452
> > +++ doc/ChangeLog 6 May 2013 09:28:56 -0000
> > @@ -1,3 +1,8 @@
> > +2013-05-06 Muhammad Bilal <mbilal@codesourcery.com>
> > +
> > + PR gdb/15224:
> > + * gdb.texinfo (Command history): Set history save enabled.
> > +
> > 2013-04-29 Tom Tromey <tromey@redhat.com>
> >
> > PR python/14204:
> > Index: NEWS
> > ===================================================================
> > RCS file: /cvs/src/src/gdb/NEWS,v
> > retrieving revision 1.593
> > diff -u -p -r1.593 NEWS
> > --- NEWS 16 Apr 2013 14:36:53 -0000 1.593
> > +++ NEWS 6 May 2013 09:30:13 -0000
> > @@ -3,6 +3,9 @@
> >
> > *** Changes since GDB 7.6
> >
> > +* The command 'history save' is now default enabled.
> > +Disabled 'history save' on testsuiltes.
> > +
> > * New commands:
> > catch rethrow
> > Like "catch throw", but catches a re-thrown exception.
> >
> >
> > Thanks,
> > -Bilal
> ping

AFAIR, you got comments, but this patch does not reflect them. Did
you send the correct diffs?