all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
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.