GNU bug report logs - #34068
26.1; next-error does not go to correct column for grep

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 34068 in the body.
You can then email your comments to 34068 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Trevor Spiteri <tspiteri <at> ieee.org>
To: bug-gnu-emacs <at> gnu.org
Subject: 26.1; next-error does not go to correct column for grep
Date: Mon, 14 Jan 2019 13:43:21 +0100

This is a regression from 25 which has the correct behaviour.

emacs -Q
C-x C-f /tmp/hello
Hello
C-x C-s
M-x grep
C-a C-k grep --color -nH --null -e ll /tmp/hello
C-x `

This puts the mark in column 0, not in column 2, even though "ll" is
highlighted in the *grep* buffer.

This is a regression in 26.1, it used to work in 25.

To fix it in my local copy, I edit grep-regexp-alist in grep.el. There
are two instances of 'grep-match-face which I replaced with an unquoted
grep-match-face.

Message #10 received at 34068-done <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Trevor Spiteri <tspiteri <at> ieee.org>
Cc: 34068-done <at> debbugs.gnu.org
Subject: Re: bug#34068: 26.1; next-error does not go to correct column for grep
Date: Sat, 19 Jan 2019 09:54:12 +0200

> From: Trevor Spiteri <tspiteri <at> ieee.org>
> Date: Mon, 14 Jan 2019 13:43:21 +0100
> 
> This is a regression from 25 which has the correct behaviour.
> 
> emacs -Q
> C-x C-f /tmp/hello
> Hello
> C-x C-s
> M-x grep
> C-a C-k grep --color -nH --null -e ll /tmp/hello
> C-x `
> 
> This puts the mark in column 0, not in column 2, even though "ll" is
> highlighted in the *grep* buffer.
> 
> This is a regression in 26.1, it used to work in 25.
> 
> To fix it in my local copy, I edit grep-regexp-alist in grep.el. There
> are two instances of 'grep-match-face which I replaced with an unquoted
> grep-match-face.

Thanks, I installed your proposed change, and I'm closing the bug.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.