GNU logs - #18379, boring messages

Message sent to bug-gnu-emacs@HIDDEN:

Subject: bug#18379: 24.3.93; Bash font lock: pretty inconsistent treatment of `` and $()
Resent-From: Carlos Pita <carlosjosepita@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: bug-gnu-emacs@HIDDEN
Resent-Date: Mon, 01 Sep 2014 14:19:02 +0000
Resent-Message-ID: <handler.18379.B.14095810916563 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
To: 18379 <at> debbugs.gnu.org
Cc: occitan@HIDDEN
From: Carlos Pita <carlosjosepita@HIDDEN>
Date: Mon, 01 Sep 2014 11:17:23 -0300
Message-ID: <87bnqz1l3g.fsf@HIDDEN>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: list
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>

The syntax table treats ` as an string quote character, so

`if [[ $x == x ]]; then echo x; else echo y fi`

$(if [[ $x == x ]]; then echo x; else echo y fi)

are highlighted in a very different way although they are semantically
equivalent.

Related to do above, I don't know whether the different highlighting of
the first word in:

$(if [[ $x == x ]]; then echo x; else echo y fi)

if [[ $x == x ]]; then echo x; else echo y fi

is intended or accidental, but it looks pretty weird. Moreover

$([[ $x == x ]] && echo x)

[[ $x == x ]] && echo x

will highlight exactly equal, which is not consistent with the previous
example.

All in all, I would suggest to:

1) Highlight $(...) and `...` the same way.

2) Don't set the face of the first symbol following $( in any special
way.

Regards
--
Carlos



In GNU Emacs 24.3.93.1 (x86_64-unknown-linux-gnu, GTK+ Version 3.12.2)
 of 2014-08-27 on carlos
Windowing system distributor `The X.Org Foundation', version 11.0.11600000
Configured using:
 `configure --prefix=/usr --sysconfdir=/etc --libexecdir=/usr/lib
 --localstatedir=/var --with-x-toolkit=gtk3 --with-xft
 'CFLAGS=-march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong
 --param=ssp-buffer-size=4' CPPFLAGS=-D_FORTIFY_SOURCE=2
 LDFLAGS=-Wl,-O1,--sort-common,--as-needed,-z,relro'

Important settings:
  value of $LC_COLLATE: en_US.UTF-8
  value of $LC_CTYPE: en_US.UTF-8
  value of $LC_MESSAGES: en_US.UTF-8
  value of $LC_MONETARY: en_US.UTF-8
  value of $LC_NUMERIC: en_US.UTF-8
  value of $LC_TIME: en_US.UTF-8
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Shell-script

Minor modes in effect:
  shell-dirtrack-mode: t
  sh-electric-here-document-mode: t
  show-paren-mode: t
  eldoc-mode: t
  yas-global-mode: t
  yas-minor-mode: t
  ido-ubiquitous-mode: t
  global-auto-complete-mode: t
  auto-complete-mode: t
  ido-everywhere: t
  winner-mode: t
  tooltip-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  column-number-mode: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
( ( x ) ) SPC & & S-SPC l s SPC " x " ) <return> $ 
( l s <backspace> <backspace> [ [ SPC $ x SPC ] ] SPC 
& & SPC l s SPC " x " ) <return> $ ( <backspace> <backspace> 
<up> C-k C-y <return> <return> C-y C-a " C-e " <up> 
<up> <down> <down> C-a C-SPC <down> M-w <down> C-y 
<up> <right> <right> <right> C-d C-d C-d C-d ( ( <right> 
<right> <right> <right> <backspace> <backspace> <backspace> 
) ) <up> <up> C-x C-s <down> <down> <down> <up> e <return> 
<backspace> <backspace> C-e <return> <up> <up> <up> 
<up> <up> <up> <up> C-SPC <down> <down> <down> <down> 
M-w <down> <return> <return> <up> <up> C-y <up> <up> 
<up> <up> C-v <down> <down> <down> <right> <up> <up> 
<up> <left> <left> C-SPC C-g <down> C-SPC <down> <down> 
<down> <right> <left> C-x r t " <return> <right> <right> 
<right> <right> <right> <right> <right> <right> <right> 
<right> <right> <right> <right> <right> <right> <right> 
<up> <up> <up> " <down> C-e " <down> C-e " <down> C-e 
" <down> <return> <return> <up> <up> <up> <up> <up> 
<up> <up> <up> C-SPC <down> M-w <down> <down> <down> 
<down> <down> <down> C-y <up> C-e <left> <left> <left> 
<left> <left> <backspace> <backspace> i f <right> <right> 
<right> <right> SPC f i <down> <up> <return> C-/ <left> 
<left> <right> <right> <backspace> <backspace> f i 
C-g <up> <down> <left> <left> <left> <right> <backspace> 
<return> <up> <down> M-^ <up> C-x C-s <down> <right> 
<right> <right> <right> <right> <right> <right> <right> 
<right> <right> <right> <right> <right> <right> <right> 
<right> <right> <right> <right> <right> ; <down> C-x 
C-s <down> <down> <up> <up> <up> C-SPC <down> M-w <down> 
<up> C-y C-a <up> C-d C-d ` C-e <backspace> ` <down> 
<down> M-x r e p o <tab> r t - e <tab> <return>

Recent messages:
[yas] Snippet 0 exited.
Undo!
Quit
Saving file /tmp/test.sh...
Wrote /tmp/test.sh
Saving file /tmp/test.sh...
Wrote /tmp/test.sh
Mark activated
Mark set
Making completion list...

Load-path shadows:
~/.emacs.d/lisp/rmail hides /usr/share/emacs/24.3.93/lisp/mail/rmail

Features:
(shadow emacsbug xterm descr-text tramp-cache tramp tramp-compat
tramp-loaddefs trampver shell view woman man misearch multi-isearch
cus-edit cus-start cus-load sh-script smie executable gnus-draft
mailalias smtpmail sendmail oauth2 warnings json plstore nnir url-http
url-gw url-auth url-queue face-remap org-colview diary-lib
diary-loaddefs cal-iso org-agenda bookmark thingatpt tabify image-file
org-capture pp rect vc-git org-element org-rmail org-mhe org-irc
org-info org-gnus org-docview doc-view jka-compr image-mode dired
org-bibtex bibtex org-bbdb org-w3m mule-util shr-color color shr
browse-url gnus-dup sort smiley gnus-cite mm-archive mail-extr
gnus-async gnus-bcklg qp gnus-ml disp-table gnus-topic nndraft nnmh
nnfolder utf-7 epa-file epa derived epg gnutls network-stream starttls
nnimap parse-time tls utf7 netrc gnus-agent gnus-srvr gnus-score
score-mode nnvirtual gnus-msg gnus-art mm-uu mml2015 epg-config mm-view
mml-smime smime dig nntp gnus-cache gnus-sum nnoo gnus-group gnus-undo
nnmail mail-source google-contacts-message google-contacts xml url-cache
url url-proxy url-privacy url-expand url-methods url-history url-cookie
url-domsuf mailcap url-util url-parse auth-source eieio eieio-core
password-cache url-vars google-oauth gnus-start gnus-spec gnus-int
gnus-range message idna rfc822 mml mml-sec mm-decode mm-bodies mm-encode
mail-parse rfc2231 rfc2047 rfc2045 ietf-drums mailabbrev gmm-utils
mailheader gnus-win gnus gnus-ems nnheader gnus-util mail-utils mm-util
mail-prsvr wid-edit server flymake compile paredit paren eldoc cl-macs
ob-python ob-R org byte-opt bytecomp byte-compile cconv org-macro
org-footnote org-pcomplete pcomplete org-list org-faces org-entities
time-date noutline outline easy-mmode org-version ob-emacs-lisp ob
ob-tangle org-src ob-ref ob-lob ob-table ob-keys ob-exp ob-comint comint
ansi-color ob-core ob-eval org-compat org-macs org-loaddefs format-spec
find-func cal-menu calendar cal-loaddefs yasnippet help-mode
ido-ubiquitous cl gv advice help-fns auto-complete-config auto-complete
popup imenu-anywhere imenu ido windmove winner ring edmacro kmacro
cl-loaddefs cl-lib info easymenu package wombat-theme tooltip electric
uniquify ediff-hook vc-hooks lisp-float-type mwheel x-win x-dnd tool-bar
dnd fontset image regexp-opt fringe tabulated-list newcomment lisp-mode
prog-mode register page menu-bar rfn-eshadow timer select scroll-bar
mouse jit-lock font-lock syntax facemenu font-core frame cham georgian
utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao korean
japanese hebrew greek romanian slovak czech european ethiopic indian
cyrillic chinese case-table epa-hook jka-cmpr-hook help simple abbrev
minibuffer nadvice loaddefs button faces cus-face macroexp files
text-properties overlay sha1 md5 base64 format env code-pages mule
custom widget hashtable-print-readable backquote make-network-process
dbusbind gfilenotify dynamic-setting system-font-setting
font-render-setting move-toolbar gtk x-toolkit x multi-tty emacs)

Memory information:
((conses 16 618712 75891)
 (symbols 48 57188 0)
 (miscs 40 3530 2211)
 (strings 32 220578 14464)
 (string-bytes 1 4703140)
 (vectors 16 51134)
 (vector-slots 8 1666305 45219)
 (floats 8 1010 1759)
 (intervals 56 28363 1382)
 (buffers 960 89)
 (heap 1024 65685 4604))

Message sent:

Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Mailer: MIME-tools 5.503 (Entity 5.503)
Content-Type: text/plain; charset=utf-8
X-Loop: help-debbugs@HIDDEN
From: help-debbugs@HIDDEN (GNU bug Tracking System)
To: Carlos Pita <carlosjosepita@HIDDEN>
Subject: bug#18379: Acknowledgement (24.3.93; Bash font lock: pretty
 inconsistent treatment of `` and $())
Message-ID: <handler.18379.B.14095810916563.ack <at> debbugs.gnu.org>
References: <87bnqz1l3g.fsf@HIDDEN>
X-Gnu-PR-Message: ack 18379
X-Gnu-PR-Package: emacs
Reply-To: 18379 <at> debbugs.gnu.org
Date: Mon, 01 Sep 2014 14:19:02 +0000

Thank you for filing a new bug report with debbugs.gnu.org.

This is an automatically generated reply to let you know your message
has been received.

Your message is being forwarded to the package maintainers and other
interested parties for their attention; they will reply in due course.

Your message has been sent to the package maintainer(s):
 bug-gnu-emacs@HIDDEN

If you wish to submit further information on this problem, please
send it to 18379 <at> debbugs.gnu.org.

Please do not send mail to help-debbugs@HIDDEN unless you wish
to report a problem with the Bug-tracking system.

--=20
18379: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=3D18379
GNU Bug Tracking System
Contact help-debbugs@HIDDEN with problems

Message sent to bug-gnu-emacs@HIDDEN:

Subject: bug#18379: 24.3.93; Bash font lock: pretty inconsistent treatment of `` and $()
Resent-From: Andreas Schwab <schwab@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: bug-gnu-emacs@HIDDEN
Resent-Date: Mon, 01 Sep 2014 15:04:02 +0000
Resent-Message-ID: <handler.18379.B18379.140958382511005 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
To: Carlos Pita <carlosjosepita@HIDDEN>
Cc: occitan@HIDDEN, 18379 <at> debbugs.gnu.org
From: Andreas Schwab <schwab@HIDDEN>
References: <87bnqz1l3g.fsf@HIDDEN>
Date: Mon, 01 Sep 2014 17:03:32 +0200
In-Reply-To: <87bnqz1l3g.fsf@HIDDEN> (Carlos Pita's message of "Mon, 01 Sep
 2014 11:17:23 -0300")
Message-ID: <mvmoauzqt6j.fsf@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Precedence: list
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>

Carlos Pita <carlosjosepita@HIDDEN> writes:

> The syntax table treats ` as an string quote character, so
>
> `if [[ $x == x ]]; then echo x; else echo y fi`
>
> $(if [[ $x == x ]]; then echo x; else echo y fi)
>
> are highlighted in a very different way although they are semantically
> equivalent.

They have different quoting rules, with `...` using rules similar to
double quotes, whereas $(...) is a fully nesting construct.

Andreas.

-- 
Andreas Schwab, SUSE Labs, schwab@HIDDEN
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE  1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."

Message sent to bug-gnu-emacs@HIDDEN:

Subject: bug#18379: 24.3.93; Bash font lock: pretty inconsistent treatment of `` and $()
Resent-From: Carlos Pita <carlosjosepita@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: bug-gnu-emacs@HIDDEN
Resent-Date: Mon, 01 Sep 2014 19:29:02 +0000
Resent-Message-ID: <handler.18379.B18379.140959969113813 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
To: Andreas Schwab <schwab@HIDDEN>
Cc: occitan@HIDDEN, 18379 <at> debbugs.gnu.org
From: Carlos Pita <carlosjosepita@HIDDEN>
References: <87bnqz1l3g.fsf@HIDDEN>
 <mailman.8037.1409583857.1147.bug-gnu-emacs@HIDDEN>
Date: Mon, 01 Sep 2014 16:27:58 -0300
In-Reply-To: <mailman.8037.1409583857.1147.bug-gnu-emacs@HIDDEN> (Andreas
 Schwab's message of "Mon, 01 Sep 2014 17:03:32 +0200")
Message-ID: <87egvv2la9.fsf@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.93 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Precedence: list
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>

Hi Andreas,

> They have different quoting rules, with `...` using rules similar to
> double quotes, whereas $(...) is a fully nesting construct.

True, but both contains bash code for a subshell which is far from being
an uninterpreted string. IMO the important distintion to highlight is
"intepreted code" vs "uninterpreted string", not the ability to nest vs
the inability to nest (due to different quoting rules).

I've been checking how other editors do this. vim works the way I'm
proposing, while scintilla based editors do it the emacs way. Regarding
my second observation before, neither vim nor scintilla treat the
leading word in a command substution in any special way, which I think
is sensible.

Cheers
--
Carlos

Message sent to bug-gnu-emacs@HIDDEN:

X-Loop: help-debbugs@HIDDEN
Subject: bug#18379: 24.3.93; Bash font lock: pretty inconsistent treatment of `` and $()
Resent-From: Stefan Monnier <monnier@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: bug-gnu-emacs@HIDDEN
Resent-Date: Wed, 03 Sep 2014 00:24:01 +0000
Resent-Message-ID: <handler.18379.B18379.140970382227498 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
X-GNU-PR-Message: followup 18379
X-GNU-PR-Package: emacs
X-GNU-PR-Keywords: 
To: Carlos Pita <carlosjosepita@HIDDEN>
Cc: occitan@HIDDEN, 18379 <at> debbugs.gnu.org
Received: via spool by 18379-submit <at> debbugs.gnu.org id=B18379.140970382227498
          (code B ref 18379); Wed, 03 Sep 2014 00:24:01 +0000
Received: (at 18379) by debbugs.gnu.org; 3 Sep 2014 00:23:42 +0000
Received: from localhost ([127.0.0.1]:57231 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.80)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1XOyMQ-00079R-0I
	for submit <at> debbugs.gnu.org; Tue, 02 Sep 2014 20:23:42 -0400
Received: from pruche.dit.umontreal.ca ([132.204.246.22]:57068)
 by debbugs.gnu.org with esmtp (Exim 4.80)
 (envelope-from <monnier@HIDDEN>) id 1XOyMM-00079G-NL
 for 18379 <at> debbugs.gnu.org; Tue, 02 Sep 2014 20:23:39 -0400
Received: from pastel.home (lechon.iro.umontreal.ca [132.204.27.242])
 by pruche.dit.umontreal.ca (8.14.1/8.14.1) with ESMTP id s830Na3m029058;
 Tue, 2 Sep 2014 20:23:36 -0400
Received: by pastel.home (Postfix, from userid 20848)
 id EA3EF6308B; Tue,  2 Sep 2014 20:23:35 -0400 (EDT)
From: Stefan Monnier <monnier@HIDDEN>
Message-ID: <jwva96h36am.fsf-monnier+emacsbugs@HIDDEN>
References: <87bnqz1l3g.fsf@HIDDEN>
Date: Tue, 02 Sep 2014 20:23:35 -0400
In-Reply-To: <87bnqz1l3g.fsf@HIDDEN> (Carlos Pita's message of "Mon, 01 Sep
 2014 11:17:23 -0300")
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.4.50 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
X-NAI-Spam-Flag: NO
X-NAI-Spam-Threshold: 5
X-NAI-Spam-Score: 0
X-NAI-Spam-Rules: 1 Rules triggered
	RV5052=0
X-NAI-Spam-Version: 2.3.0.9378 : core <5052> : inlines <1229> : streams
 <1279958> : uri <1809668>
X-Spam-Score: -3.0 (---)
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <http://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <http://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <http://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -3.0 (---)

> The syntax table treats ` as an string quote character, so
>
> `if [[ $x == x ]]; then echo x; else echo y fi`
> $(if [[ $x == x ]]; then echo x; else echo y fi)
>
> are highlighted in a very different way although they are semantically
> equivalent.

This is largely historical, I think (with the $(...) highlighting
appearing more recently).  I don't have a strong opinion either way, but
highlighting it differently could be construed as a feature as well
(letting the user choose between the two quoting mechanisms based on
how she likes it to be highlighted).

IOW we could change it, but the motivation to do so is not very high and
it's not "obviously better".

> Related to do above, I don't know whether the different highlighting of
> the first word in:
>
> $(if [[ $x == x ]]; then echo x; else echo y fi)
> if [[ $x == x ]]; then echo x; else echo y fi
>
> is intended or accidental, but it looks pretty weird.

I don't know either, but I agree it's weird.  Patch welcome.


        Stefan

Last modified: Mon, 25 Nov 2019 12:00:02 UTC

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997 nCipher Corporation Ltd, 1994-97 Ian Jackson.