Ubuntu
gtksourceview3 package

perl syntax highlighting doesn't work for "-s"

Bug #631856 reported by Saddy on 2010-09-06

This bug affects 2 people

Affects		Status	Importance	Assigned to	Milestone
	GtkSourceView	Invalid	Low	gnome-bugs #655536
	gtksourceview3 (Ubuntu)	Triaged	Low	Unassigned

Bug Description

Binary package hint: gedit

Code:

while ( (not -e "$omg") or (-s "$omg" < 71) ) {
getstore($img, $omg);
}

Everything after (-s "$omg" < 71) gets highlighted even though the code is correct.

ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: gedit 2.30.3-0ubuntu0.1
ProcVersionSignature: Ubuntu 2.6.32-24.41-generic 2.6.32.15+drm33.5
Uname: Linux 2.6.32-24-generic x86_64
NonfreeKernelModules: nvidia
Architecture: amd64
Date: Mon Sep 6 21:08:38 2010
ProcEnviron:
LANG=de_DE.UTF-8
SHELL=/bin/bash
SourcePackage: gedit

Tags:

Revision history for this message

Saddy (sadmail) wrote on 2010-09-06:

Dependencies.txt Edit (4.6 KiB, text/plain; charset="utf-8")

Revision history for this message

Pedro Villavicencio (pedro) wrote on 2010-09-08:

thanks for the report, could you take an screenshot of what you're seeing there? i'm getting exactly the same behavior with vim, emacs and gedit.

Changed in gedit (Ubuntu):
importance:	Undecided → Low
status:	New → Incomplete

Revision history for this message

Saddy (sadmail) wrote on 2010-09-09: Re: [Bug 631856] Re: perl syntax highlighting doesn't work for "-s"

Bildschirmfoto-18.png Edit (73.7 KiB, image/png; name="Bildschirmfoto-18.png")

Am 08.09.2010 18:50, schrieb Pedro Villavicencio:
> thanks for the report, could you take an screenshot of what you're
> seeing there? i'm getting exactly the same behavior with vim, emacs and
> gedit.
>
> ** Changed in: gedit (Ubuntu)
> Importance: Undecided => Low
>
> ** Changed in: gedit (Ubuntu)
> Status: New => Incomplete
>
>
Yup. E.g. the 'foreach' should be marked in red after the '-s' position.

Revision history for this message

Pedro Villavicencio (pedro) wrote on 2010-09-09:

not confirming, could you attach that file or a piece of code showing the failure? thanks.

Revision history for this message

McFly81 (christian-lange-81) wrote on 2010-12-13:

test.pl Edit (315 bytes, text/plain)

Confirming this on gedit 2.30.3 (Ubuntu 10.04 LTS). I wrote a few lines (only as example) from Saddys screenshot into a pl-file and gedit highlights it wrong. I will attach the the sourcefile.

Revision history for this message

Andreas Moog (ampelbein) wrote on 2011-03-03:

Screenshot.png Edit (140.2 KiB, image/png)

This is fixed in the development release, see attached screen.

Changed in gedit (Ubuntu):
status:	Incomplete → Fix Released
status:	Fix Released → Confirmed

Revision history for this message

Andreas Moog (ampelbein) wrote on 2011-03-03:

meant to say "not fixed", sorry for the spam.

Bug Watch Updater (bug-watch-updater) on 2012-02-08

Changed in gedit:
importance:	Unknown → Low
status:	Unknown → New

Sebastien Bacher (seb128) on 2012-02-08

Changed in gedit (Ubuntu):
status:	Confirmed → Triaged

Revision history for this message

Mysha (mysha) wrote on 2012-02-11:

Usually, a few countermeasures can reduce this to end of the line only.
E.g., from one of my scripts:

$grutte= -s( "$bank.fb"); #; #

But indeed, it's irritating to have the text mishighlighting. I've been looking at where we can customize the language highlighting, but I can't find it. So, as I read whole file contents a lot, and therefor need to get the file size, this is to me an ever present annoyance.

Revision history for this message

nmz (s-namitha-s) wrote on 2012-06-06:

diff_perl.txt Edit (676 bytes, text/plain)

Attached a proposed patch

Sebastien Bacher (seb128) on 2012-06-06

affects:	gedit (Ubuntu) → gtksourceview3 (Ubuntu)
affects:	gedit → gtksourceview

Revision history for this message

Ubuntu Foundations Team Bug Bot (crichton) wrote on 2012-06-06:

#10

The attachment "diff_perl.txt" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags:

added: patch

Bug Watch Updater (bug-watch-updater) on 2013-03-30