GNU bug report logs - #5719
[patch] fix bat-generic-mode highlighting pattern for CLI switch.

Previous Next

Package: emacs;

Reported by: Oleksandr Gavenko <gavenkoa <at> gmail.com>

Date: Sun, 14 Mar 2010 20:19:01 UTC

Severity: normal

Tags: patch

Done: Chong Yidong <cyd <at> stupidchicken.com>

Bug is archived. No further changes may be made.

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

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

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to owner <at> debbugs.gnu.org, bug-gnu-emacs <at> gnu.org:
bug#5719; Package emacs. (Sun, 14 Mar 2010 20:19:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Oleksandr Gavenko <gavenkoa <at> gmail.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Sun, 14 Mar 2010 20:19:01 GMT) Full text and rfc822 format available.

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

From: Oleksandr Gavenko <gavenkoa <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: [patch] fix bat-generic-mode highlighting pattern for CLI switch.
Date: Sun, 14 Mar 2010 22:18:19 +0200
[Message part 1 (text/plain, inline)]
Pattern for highlight '/help', '/query', '/a' like
command line options currently does not require
leading whitespace char but MUST.

If you use Cygwin/MSYS from bat file you can type expressions like

  ls dir/sub-dir

and '/sub-dir' unfortunately highlighted as CLI switch!

Switch which start with - or + properly required leading whitespaces.

There exist one situation then you must highlight switch
exactly at start of line (and so there are no any leading whitespaces)
because this line considered continuation of previous line
if previous line end with '^' char.

But this not true for command.com and so unusual situation
so we can ignore it.

So in patch I just improve pattern for '+'/'-', which already
do job right.

-- 
Best regards!
[.patch (text/plain, attachment)]

Reply sent to Chong Yidong <cyd <at> stupidchicken.com>:
You have taken responsibility. (Sun, 27 Jun 2010 18:45:02 GMT) Full text and rfc822 format available.

Notification sent to Oleksandr Gavenko <gavenkoa <at> gmail.com>:
bug acknowledged by developer. (Sun, 27 Jun 2010 18:45:03 GMT) Full text and rfc822 format available.

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

From: Chong Yidong <cyd <at> stupidchicken.com>
To: Oleksandr Gavenko <gavenkoa <at> gmail.com>
Cc: 5719-done <at> debbugs.gnu.org
Subject: Re: [patch] fix bat-generic-mode highlighting pattern for CLI switch.
Date: Sun, 27 Jun 2010 14:44:46 -0400
> Pattern for highlight '/help', '/query', '/a' like command line
> options currently does not require leading whitespace char but MUST.

Thanks for the patch.  I've checked it in.





bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Mon, 26 Jul 2010 11:24:04 GMT) Full text and rfc822 format available.

This bug report was last modified 15 years and 24 days ago.

Previous Next


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