Ticket #2142 (new defect)

Opened 9 years ago

Last modified 10 months ago

mcedit highlighting stumbles upon multiple nested quotes

Reported by: zaytsev Owned by:
Priority: minor Milestone: Future Releases
Component: mcedit Version: master
Keywords: Cc: ksquirrel.iv@…, gotar@…, Curtis@…, muzzol, vitalif@…, polynomial-c@…, teresaejunior@…, xmlich02@…
Blocked By: Blocking: #2145, #2224, #2359
Branch state: no branch Votes for changeset:

Description

Forwarded from Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=577929

From: Dmitry Baryshev <ksquirrel.iv@gmail.com>
Subject: mcedit colorer problem
Date: Thu, 15 Apr 2010 13:38:59 +0300

If I type the following string in a shell script:

valid=$(echo "$device" | awk '{ match($0, /^\/dev\/sd[a-z][0-9]$/); print
RSTART }')

mcedit will color all the following text as a character string (screenshot
is attached)

-- 
Regards, Krasu

Attachments

снимок1.png (2.9 KB) - added by zaytsev 9 years ago.
highlighting.png (2.0 KB) - added by asy 5 years ago.
another example

Change History

Changed 9 years ago by zaytsev

comment:1 Changed 8 years ago by gotar

  • Cc gotar@… added

comment:2 Changed 8 years ago by curtis

  • Cc Curtis@… added

comment:3 follow-up: ↓ 5 Changed 8 years ago by angel_il

curtis: Currently, nobody working on the engine syntax :( this mean what this ticket will be not closet long-long time...

comment:4 Changed 7 years ago by slavazanko

  • Cc muzzol added

comment:5 in reply to: ↑ 3 ; follow-up: ↓ 6 Changed 7 years ago by muzzol

  • Branch state set to no branch

Replying to angel_il:

curtis: Currently, nobody working on the engine syntax :( this mean what this ticket will be not closet long-long time...

not sure if this has been discussed before, but why not include colorer take5 to make syntax work?

is there any problem providing it with MC?

i know lot of people is compiling it by themselves so is just a matter of polishing makefile.

comment:6 in reply to: ↑ 5 Changed 7 years ago by andrew_b

Replying to muzzol:

not sure if this has been discussed before, but why not include colorer take5 to make syntax work?

First of all, colorer-take5 is written in C++...
Then, the any extra library is not nice in embedded system.

comment:7 Changed 7 years ago by vitalif

  • Cc vitalif@… added

comment:8 Changed 7 years ago by andrew_b

  • Milestone changed from 4.7 to Future Releases

comment:9 Changed 6 years ago by andrew_b

  • Blocking 2145, 2224, 2359 added

comment:10 Changed 5 years ago by Polynomial-C

  • Cc polynomial-c@… added

comment:11 Changed 5 years ago by andrew_b

Ticket #3129 has been marked as a duplicate of this ticket.

Changed 5 years ago by asy

another example

comment:12 Changed 5 years ago by asy

A work around for code:

LINES=($(egrep -v "^(#|[[:space:]])|^$" $HOME/file))

#" <- work arround

for LINE in "${LINES[@]}"; do
    echo "--" $LINE "--"
done

comment:13 Changed 4 years ago by andrew_b

Ticket #3213 has been marked as a duplicate of this ticket.

comment:14 Changed 4 years ago by andrew_b

Ticket #3377 has been marked as a duplicate of this ticket.

comment:15 Changed 2 years ago by teresaejunior

This bug is truly annoying! Almost any shell script with some regex foo gets broken in mcedit, maybe fixing #2931 would be the best long term decision for fixing this!

comment:16 Changed 2 years ago by teresaejunior

  • Cc teresaejunior@… added

comment:17 Changed 11 months ago by andrew_b

Ticket #3881 has been marked as a duplicate of this ticket.

comment:18 Changed 10 months ago by xmlich02

  • Cc xmlich02@… added
Note: See TracTickets for help on using tickets.