From: LanX <lanx.perl@googlemail.com>
To: help-gnu-emacs@gnu.org
Subject: [cperl-mode] minor highlighting deficiencies
Date: Thu, 27 May 2010 04:19:16 -0700 (PDT) [thread overview]
Message-ID: <0f24d932-0dd3-4c9b-8d5b-da3268f2bc14@y21g2000vba.googlegroups.com> (raw)
Hi
1) UNITCHECK blocks aren't treated like CHECK blocks i.e. `font-lock-
keyword-face'
2) Highlighting of variables in strings are not consistent
a) in $a="@a %h $s";
the arrays and hashes are highlighted the scalars are not (`font-
lock-string-face')
b) there is no difference between single and double quoted
(interpolated) strings.
$a='@a %h $s';
c) escaping doesnt change the highlighting of arrays and hashes
$a="\@a \%h \$s";
3) Variablegrouping in braces are not discovered
print @{a} %{h} ${s};
while it's disputable if the identifieres within the braces are
easily parsed,
since @{$array_ref} is a dereferencing command, it could be at
least suggested
to highlight the surrounding @{ ...} in cperl-array-face or at
least
perl-nonoverridable-face like in constructs like qq{...}
4) perl allows whitespaces between sigils and identifier, i.e. @ a and
@ {a} are legal
arrays ... while I wouldn't say it's good style.
All of these suggestions except the first one need quite complicated
look around parsings, as a minimal solution it should be considered at
least to skip variable highlighting within strings, like they are
ignored in scalar variables.
Cheers
Rolf
reply other threads:[~2010-05-27 11:19 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=0f24d932-0dd3-4c9b-8d5b-da3268f2bc14@y21g2000vba.googlegroups.com \
--to=lanx.perl@googlemail.com \
--cc=help-gnu-emacs@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.