GNU bug report logs - #23391
24.5; javascript indentation inconsistency

Previous Next

Package: emacs;

Reported by: Paul Becker <raincomplex <at> gmail.com>

Date: Wed, 27 Apr 2016 21:05:02 UTC

Severity: normal

Found in version 24.5

Fixed in version 25.1

Done: Dmitry Gutov <dgutov <at> yandex.ru>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 23391 in the body.
You can then email your comments to 23391 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#23391; Package emacs. (Wed, 27 Apr 2016 21:05:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Paul Becker <raincomplex <at> gmail.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Wed, 27 Apr 2016 21:05:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Paul Becker <raincomplex <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.5; javascript indentation inconsistency
Date: Wed, 27 Apr 2016 16:54:13 -0400
Started with emacs -Q. Having a space after the "for" (see below) messes 
up the indentation on the rest of the items in the list. Without a space 
between the "for" and the open paren, the indentation is normal.

// no space between "for" and "("
[
    function(){
        for(){
        }
    },
    function(){},
];

// space between "for" and "("
[
    function(){
        for (){
        }
    },
        function(){},  // everything from here on is incorrectly indented
        ];



In GNU Emacs 24.5.1 (x86_64-unknown-linux-gnu, GTK+ Version 3.16.6)
 of 2015-09-09 on foutrelis
Windowing system distributor `The X.Org Foundation', version 11.0.11803000
System Description:    Arch Linux

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 $LANG: en_US.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Javascript

Minor modes in effect:
  tooltip-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Recent messages:
Indenting region...done
Mark set
Indenting region...done
Mark set
Indenting region...done
Mark set
Indenting region...done
Mark set
Indenting region...done
End of buffer [2 times]
call-interactively: End of buffer [6 times]

Load-path shadows:
None found.

Features:
(js advice byte-opt bytecomp byte-compile cl-extra cl-loaddefs cl-lib
cconv json imenu thingatpt cc-mode cc-fonts cc-guess cc-menus cc-cmds
cc-styles cc-align cc-engine cc-vars cc-defs help-mode pp shadow sort
gnus-util mail-extr emacsbug message format-spec rfc822 mml easymenu
mml-sec mm-decode mm-bodies mm-encode mail-parse rfc2231 mailabbrev
gmm-utils mailheader sendmail rfc2047 rfc2045 ietf-drums mm-util
help-fns mail-prsvr mail-utils time-date 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 104164 6721)
 (symbols 48 20876 0)
 (miscs 40 50 228)
 (strings 32 19040 5822)
 (string-bytes 1 662599)
 (vectors 16 13604)
 (vector-slots 8 428231 7582)
 (floats 8 68 295)
 (intervals 56 287 17)
 (buffers 960 13)
 (heap 1024 26252 889))





Reply sent to Dmitry Gutov <dgutov <at> yandex.ru>:
You have taken responsibility. (Wed, 27 Apr 2016 23:05:02 GMT) Full text and rfc822 format available.

Notification sent to Paul Becker <raincomplex <at> gmail.com>:
bug acknowledged by developer. (Wed, 27 Apr 2016 23:05:02 GMT) Full text and rfc822 format available.

Message #10 received at 23391-done <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Paul Becker <raincomplex <at> gmail.com>, 23391-done <at> debbugs.gnu.org
Subject: Re: bug#23391: 24.5; javascript indentation inconsistency
Date: Thu, 28 Apr 2016 02:03:57 +0300
Version: 25.1

On 04/27/2016 11:54 PM, Paul Becker wrote:
> Started with emacs -Q. Having a space after the "for" (see below) messes
> up the indentation on the rest of the items in the list. Without a space
> between the "for" and the open paren, the indentation is normal.

Thanks for the report, should be fixed now. It was mistaking this array 
of functions for an array comprehension.




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Thu, 26 May 2016 11:24:05 GMT) Full text and rfc822 format available.

This bug report was last modified 7 years and 308 days ago.

Previous Next


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