Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mcedit highlighting stumbles upon multiple nested quotes #2142

Open
mc-butler opened this issue Apr 15, 2010 · 24 comments
Open

mcedit highlighting stumbles upon multiple nested quotes #2142

mc-butler opened this issue Apr 15, 2010 · 24 comments
Labels
area: mcedit mcedit, the built-in text editor prio: low Minor problem or easily worked around

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/2142
Reporter zaytsev (@zyv)
Mentions ksquirrel.iv@….com, gotar@….pl, Curtis@….net (@dotysan), muzzol, vitalif@….ru, polynomial-c@….de, teresaejunior@….com, xmlich02@….fit.vutbr.cz

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

Note

Original attachments:

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Apr 15, 2010 at 11:53 UTC

@mc-butler
Copy link
Author

Changed by gotar (gotar@….pl) on May 26, 2010 at 20:22 UTC (comment 1)

  • Cc changed from ksquirrel.iv@….com to ksquirrel.iv@….com, gotar@….pl

@mc-butler
Copy link
Author

Changed by curtis (@dotysan) on Feb 13, 2011 at 18:46 UTC (comment 2)

  • Cc changed from ksquirrel.iv@….com, gotar@….pl to ksquirrel.iv@….com, gotar@….pl, Curtis@….net

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Feb 13, 2011 at 19:52 UTC (comment 3)

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

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Jul 5, 2011 at 7:09 UTC (comment 4)

  • Cc changed from ksquirrel.iv@….com, gotar@….pl, Curtis@….net to ksquirrel.iv@….com, gotar@….pl, Curtis@….net, muzzol

@mc-butler
Copy link
Author

Changed by muzzol on Jul 26, 2011 at 9:57 UTC (comment 3.5)

  • 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.

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Jul 26, 2011 at 10:04 UTC (comment 5.6)

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.

@mc-butler
Copy link
Author

Changed by vitalif (vitalif@….ru) on Oct 11, 2011 at 10:56 UTC (comment 7)

  • Cc changed from ksquirrel.iv@….com, gotar@….pl, Curtis@….net, muzzol to ksquirrel.iv@….com, gotar@….pl, Curtis@….net, muzzol, vitalif@….ru

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Nov 3, 2011 at 9:23 UTC (comment 8)

  • Milestone changed from 4.7 to Future Releases

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Jul 21, 2012 at 5:47 UTC (comment 9)

@mc-butler
Copy link
Author

Changed by Polynomial-C (polynomial-c@….de) on Jun 21, 2013 at 10:22 UTC (comment 10)

  • Cc changed from ksquirrel.iv@….com, gotar@….pl, Curtis@….net, muzzol, vitalif@….ru to ksquirrel.iv@….com, gotar@….pl, Curtis@….net, muzzol, vitalif@….ru, polynomial-c@….de

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 27, 2013 at 14:07 UTC (comment 11)

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

@mc-butler
Copy link
Author

Changed by asy (a_s_y@….ru) on Apr 2, 2014 at 5:35 UTC

another example

@mc-butler
Copy link
Author

Changed by asy (a_s_y@….ru) on Apr 2, 2014 at 5:39 UTC (comment 12)

A work around for code:

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

#" <- work arround

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

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on May 14, 2014 at 4:04 UTC (comment 13)

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

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 12, 2014 at 5:21 UTC (comment 14)

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

@mc-butler
Copy link
Author

Changed by teresaejunior (teresaejunior@….com) on May 16, 2016 at 0:20 UTC (comment 15)

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!

@mc-butler
Copy link
Author

Changed by teresaejunior (teresaejunior@….com) on May 16, 2016 at 0:25 UTC (comment 16)

  • Cc changed from ksquirrel.iv@….com, gotar@….pl, Curtis@….net, muzzol, vitalif@….ru, polynomial-c@….de to ksquirrel.iv@….com, gotar@….pl, Curtis@….net, muzzol, vitalif@….ru, polynomial-c@….de, teresaejunior@….com

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Nov 27, 2017 at 17:10 UTC (comment 17)

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

@mc-butler
Copy link
Author

Changed by xmlich02 (xmlich02@….fit.vutbr.cz) on Dec 4, 2017 at 12:59 UTC (comment 18)

  • Cc changed from ksquirrel.iv@….com, gotar@….pl, Curtis@….net, muzzol, vitalif@….ru, polynomial-c@….de, teresaejunior@….com to ksquirrel.iv@….com, gotar@….pl, Curtis@….net, muzzol, vitalif@….ru, polynomial-c@….de, teresaejunior@….com, xmlich02@….fit.vutbr.cz

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Jan 14, 2020 at 17:40 UTC (comment 19)

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

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Feb 29, 2020 at 6:36 UTC (comment 20)

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

@mc-butler
Copy link
Author

Changed by mussol (@muzzol) on Jun 15, 2021 at 22:55 UTC (comment 21)

anyone working on this?

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

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Jun 20, 2021 at 12:24 UTC (comment 22)

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.

@mc-butler mc-butler marked this as a duplicate of #2498 Feb 28, 2025
@mc-butler mc-butler marked this as a duplicate of #2568 Feb 28, 2025
@mc-butler mc-butler marked this as a duplicate of #3129 Feb 28, 2025
@mc-butler mc-butler marked this as a duplicate of #3213 Feb 28, 2025
@mc-butler mc-butler marked this as a duplicate of #3377 Feb 28, 2025
@mc-butler mc-butler marked this as a duplicate of #3881 Feb 28, 2025
@mc-butler mc-butler marked this as a duplicate of #4048 Feb 28, 2025
@mc-butler mc-butler marked this as a duplicate of #4069 Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: mcedit mcedit, the built-in text editor prio: low Minor problem or easily worked around
Development

No branches or pull requests

1 participant