Ticket #2142 (new defect)

Opened 14 years ago

Last modified 3 years 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 14 years ago.
highlighting.png (2.0 KB) - added by asy 11 years ago.
another example

Change History

Changed 14 years ago by zaytsev

comment:1 Changed 14 years ago by gotar

  • Cc gotar@… added

comment:2 Changed 14 years ago by curtis

  • Cc Curtis@… added

comment:3 follow-up: ↓ 5 Changed 14 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 13 years ago by slavazanko

  • Cc muzzol added

comment:5 in reply to: ↑ 3 ; follow-up: ↓ 6 Changed 13 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 13 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 13 years ago by vitalif

  • Cc vitalif@… added

comment:8 Changed 13 years ago by andrew_b

  • Milestone changed from 4.7 to Future Releases

comment:9 Changed 12 years ago by andrew_b

  • Blocking 2145, 2224, 2359 added

comment:10 Changed 11 years ago by Polynomial-C

  • Cc polynomial-c@… added

comment:11 Changed 11 years ago by andrew_b

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

Changed 11 years ago by asy

another example

comment:12 Changed 11 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 10 years ago by andrew_b

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

comment:14 Changed 10 years ago by andrew_b

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

comment:15 Changed 8 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 8 years ago by teresaejunior

  • Cc teresaejunior@… added

comment:17 Changed 7 years ago by andrew_b

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

comment:18 Changed 7 years ago by xmlich02

  • Cc xmlich02@… added

comment:19 Changed 5 years ago by andrew_b

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

comment:20 Changed 5 years ago by andrew_b

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

comment:21 Changed 3 years ago by mussol

anyone working on this?

I see some love done on syntax highlighting on last versions, it would be nice to solve this particular issue.

comment:22 Changed 3 years ago by zaytsev

Nope, there is a fork trying to cram new features in mcedit and produce new bugs, but there is sadly no interest from contributors to fix existing bugs. The maintainers do not have resources to address this particular one - it requires a lot of time.

Note: See TracTickets for help on using tickets.