[Emacs-diffs] /srv/bzr/emacs/trunk r111516: Fix interpretation of gnu line.col1-col2 error message format

[Glenn Morris]

------------------------------------------------------------
revno: 111516
fixes bug: http://debbugs.gnu.org/13335
committer: Glenn Morris <address@hidden>
branch nick: trunk
timestamp: Sun 2013-01-13 17:08:13 -0800
message:
  Fix interpretation of gnu line.col1-col2 error message format
  
  * lisp/progmodes/compile.el (compilation-error-regexp-alist-alist):
  * test/automated/compile-tests.el (compile-tests--test-regexps-data):
  Fix interpretation of gnu line.col1-col2 format.
modified:
  lisp/ChangeLog
  lisp/progmodes/compile.el
  test/ChangeLog
  test/automated/compile-tests.el

=== modified file 'lisp/ChangeLog'
--- a/lisp/ChangeLog    2013-01-13 20:03:01 +0000
+++ b/lisp/ChangeLog    2013-01-14 01:08:13 +0000
@@ -1,3 +1,8 @@
+2013-01-14  Glenn Morris  <address@hidden>
+
+       * progmodes/compile.el (compilation-error-regexp-alist-alist):
+       Fix interpretation of gnu line.col1-col2 format.  (Bug#13335)
+
 2013-01-13  Fabián Ezequiel Gallina  <address@hidden>
 
        * progmodes/python.el (python-nav-end-of-statement): Fix

=== modified file 'lisp/progmodes/compile.el'
--- a/lisp/progmodes/compile.el 2013-01-11 23:08:55 +0000
+++ b/lisp/progmodes/compile.el 2013-01-14 01:08:13 +0000
@@ -134,6 +134,7 @@
 
 ;; If you make any changes to `compilation-error-regexp-alist-alist',
 ;; be sure to run the ERT test in test/automated/compile-tests.el.
+;; emacs -batch -l compile-tests.el -f ert-run-tests-batch-and-exit
 
 (defvar compilation-error-regexp-alist-alist
   '((absoft
@@ -261,8 +262,8 @@
      ;; The "in \\|from " exception was added to handle messages from Ruby.
      "^\\(?:[[:alpha:]][-[:alnum:].]+: ?\\|[ \t]+\\(?:in \\|from \\)\\)?\
 \\([0-9]*[^0-9\n]\\(?:[^\n :]\\| [^-/\n]\\|:[^ \n]\\)*?\\): ?\
-\\([0-9]+\\)\\(?:[.:]\\([0-9]+\\)\\)?\
-\\(?:-\\([0-9]+\\)?\\(?:\\.\\([0-9]+\\)\\)?\\)?:\
+\\([0-9]+\\)\\(?:-\\(?4:[0-9]+\\)\\(?:\\.\\(?5:[0-9]+\\)\\)?\
+\\|[.:]\\(?3:[0-9]+\\)\\(?:-\\(?:\\(?4:[0-9]+\\)\\.\\)?\\(?5:[0-9]+\\)\\)?\\)?:\
 \\(?: *\\(\\(?:Future\\|Runtime\\)?[Ww]arning\\|W:\\)\\|\
  *\\([Ii]nfo\\(?:\\>\\|rmationa?l?\\)\\|I:\\|instantiated from\\|[Nn]ote\\)\\|\
  *[Ee]rror\\|\[0-9]?\\(?:[^0-9\n]\\|$\\)\\|[0-9][0-9][0-9]\\)"

=== modified file 'test/ChangeLog'
--- a/test/ChangeLog    2013-01-10 15:01:35 +0000
+++ b/test/ChangeLog    2013-01-14 01:08:13 +0000
@@ -1,3 +1,8 @@
+2013-01-14  Glenn Morris  <address@hidden>
+
+       * automated/compile-tests.el (compile-tests--test-regexps-data):
+       Fix interpretation of gnu line.col1-col2 format.  (Bug#13335)
+
 2013-01-10  Wolfgang Jenkner  <address@hidden>
 
        * automated/man-tests.el: New file.

=== modified file 'test/automated/compile-tests.el'
--- a/test/automated/compile-tests.el   2013-01-01 09:11:05 +0000
+++ b/test/automated/compile-tests.el   2013-01-14 01:08:13 +0000
@@ -176,8 +176,10 @@
     ("foo.c:8.23: note: message" 1 23 8 "foo.c")
     ("foo.c:8.23: info: message" 1 23 8 "foo.c")
     ("foo.c:8:23:information: message" 1 23 8 "foo.c")
-    ("foo.c:8.23-45: Informational: message" 1 (23 . nil) (8 . 45) "foo.c")
+    ("foo.c:8.23-45: Informational: message" 1 (23 . 46) (8 . nil) "foo.c")
     ("foo.c:8-23: message" 1 nil (8 . 23) "foo.c")
+    ;; The next one is not in the GNU standards AFAICS.
+    ;; Here we seem to interpret it as LINE1-LINE2.COL2.
     ("foo.c:8-45.3: message" 1 (nil . 4) (8 . 45) "foo.c")
     ("foo.c:8.23-9.1: message" 1 (23 . 2) (8 . 9) "foo.c")
     ("jade:dbcommon.dsl:133:17:E: missing argument for function call"