bug#23391: 24.5; javascript indentation inconsistency

bug#23391: 24.5; javascript indentation inconsistency

[Paul Becker]

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))

bug#23391: 24.5; javascript indentation inconsistency

[Dmitry Gutov]

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.