Patch: fixlets to dwarf-mode.el

Tom Tromey tromey@redhat.com
Tue Jun 5 20:41:00 GMT 2012


This patch fixes a few minor issues in dwarf-mode.el.

First, it adds a trailing comment.  This lets you install the file using
package.el, e.g., M-x package-install-file.

Next, it fixes a byte compilation warning found when doing this.

Finally, it bumps the version number, which I think is a nice thing to
do whenever modifying it.  In most cases this is needed so that
package.el will install the updated version; in this particular case
that could be avoided (since one cannot install the unpatched version),
but I felt it was a good practice to get into.

Ok?

Tom

2012-06-05  Tom Tromey  <tromey@redhat.com>

	* dwarf-mode.el: Add final comment.  Bump version.
	(dwarf-insert-substructure-button): Use string-to-number.

Index: dwarf-mode.el
===================================================================
RCS file: /cvs/src/src/binutils/dwarf-mode.el,v
retrieving revision 1.2
diff -u -r1.2 dwarf-mode.el
--- dwarf-mode.el	16 Jun 2011 16:39:23 -0000	1.2
+++ dwarf-mode.el	5 Jun 2012 20:40:34 -0000
@@ -1,6 +1,6 @@
 ;;; dwarf-mode.el --- Browser for DWARF information.
 
-;; Version: 1.0
+;; Version: 1.1
 
 ;; This file is not part of GNU Emacs, but is distributed under the
 ;; same terms:
@@ -56,7 +56,7 @@
   (beginning-of-line)
   (unless (looking-at "^ <\\([0-9]+\\)>")
     (error "Unrecognized line."))
-  (let ((new-depth (1+ (string-to-int (match-string 1)))))
+  (let ((new-depth (1+ (string-to-number (match-string 1)))))
     (dwarf-do-insert-substructure new-depth die)))
 
 (defun dwarf-insert-substructure (arg)
@@ -165,3 +165,5 @@
     (dwarf-do-refresh)))
 
 (provide 'dwarf-mode)
+
+;;; dwarf-mode.el ends here



More information about the Binutils mailing list