Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: mbilal <mbilal@codesourcery.com>
To: <gdb-patches@sourceware.org>
Cc: <tromey@redhat.com>
Subject: Fwd: [PATCH] Fix for PR gdb/15224  should "set history save on" by default
Date: Fri, 29 Mar 2013 16:18:00 -0000	[thread overview]
Message-ID: <51558EC5.2030806@codesourcery.com> (raw)
In-Reply-To: <51558CB4.8010003@codesourcery.com>

Hi All,
The following patch is a fix for PR gdb/15224, that is 'history save'
command should be enabled by default
so that user can always be able to see their commands history by default

Index: gdb/top.c
===================================================================
RCS file: /cvs/src/src/gdb/top.c,v
retrieving revision 1.231
diff -u -p -r1.231 top.c
--- gdb/top.c    27 Mar 2013 12:14:09 -0000    1.231
+++ gdb/top.c    29 Mar 2013 11:39:40 -0000
@@ -1604,7 +1604,7 @@ init_main (void)
    /* Set the important stuff up for command editing.  */
    command_editing_p = 1;
    history_expansion_p = 0;
-  write_history_p = 0;
+  write_history_p = 1;

    /* Setup important stuff for command line editing.  */
    rl_completion_word_break_hook = gdb_completion_word_break_characters;
Index: gdb/NEWS
===================================================================
RCS file: /cvs/src/src/gdb/NEWS,v
retrieving revision 1.577
diff -u -p -r1.577 NEWS
--- gdb/NEWS    28 Mar 2013 21:58:03 -0000    1.577
+++ gdb/NEWS    29 Mar 2013 11:41:42 -0000
@@ -3,6 +3,8 @@

  *** Changes since GDB 7.6

+* The command 'history save' is now default enabled
+
  * New commands:
  maint set|show per-command
  maint set|show per-command space
Index: gdb/ChangeLog
===================================================================
RCS file: /cvs/src/src/gdb/ChangeLog,v
retrieving revision 1.15339
diff -u -p -r1.15339 ChangeLog
--- gdb/ChangeLog    29 Mar 2013 02:04:14 -0000    1.15339
+++ gdb/ChangeLog    29 Mar 2013 11:43:27 -0000
@@ -1,3 +1,10 @@
+2013-03-29  Muhammad Bilal  <mbilal@codesourcery.com>
+
+     PR gdb/15224
+     * top.c (init_main):Modified write_history_p.
+
+     * NEWS: Update.
+
  2013-03-28  Joel Brobecker  <brobecker@adacore.com>

      * coffread.c (is_import_fixup_symbol): New function.
mbilal@pkl-mbilal-ubuntu:~/src/gdb/doc$ cvs diff gdb.texinfo
Index: gdb.texinfo
===================================================================
RCS file: /cvs/src/src/gdb/doc/gdb.texinfo,v
retrieving revision 1.1067
diff -u -p -r1.1067 gdb.texinfo
--- gdb.texinfo    28 Mar 2013 21:58:03 -0000    1.1067
+++ gdb.texinfo    29 Mar 2013 12:22:31 -0000
@@ -21218,7 +21218,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: gdb/doc/ChangeLog
===================================================================
RCS file: /cvs/src/src/gdb/doc/ChangeLog,v
retrieving revision 1.1430
diff -u -p -r1.1430 ChangeLog
--- gdb/doc/ChangeLog    28 Mar 2013 21:58:03 -0000    1.1430
+++ gdb/doc/ChangeLog    29 Mar 2013 11:51:07 -0000
@@ -1,3 +1,8 @@
+2013-03-29  Muhammad Bilal  <mbilal@codesourcery.com>
+
+    PR gdb/15224
+    * gdb.texinfo (Command history): Set history save enabled
+
  2013-03-28  Pedro Alves  <palves@redhat.com>

      * gdb.texinfo (Remote Configuration) <set remote @var{name}-packet


Ok to Commit?


Thanks
-Bilal




       reply	other threads:[~2013-03-29 12:53 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <51558CB4.8010003@codesourcery.com>
2013-03-29 16:18 ` mbilal [this message]
2013-03-29 16:28   ` Jan Kratochvil
2013-04-01 11:34     ` Pedro Alves
2013-04-01 12:41       ` Jan Kratochvil
2013-04-01 19:44         ` Pedro Alves
2013-04-02 17:10           ` Jan Kratochvil
2013-04-03 17:14             ` Pedro Alves
2013-04-03 17:50               ` Jan Kratochvil

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51558EC5.2030806@codesourcery.com \
    --to=mbilal@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    --cc=tromey@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox