[ESS-bugs] Variable name affects indentation

Vitalie Spinu spinuvit at gmail.com
Sat Aug 18 12:28:48 CEST 2012


  >> Martin Maechler <maechler at stat.math.ethz.ch>
  >> on Sat, 18 Aug 2012 11:52:35 +0200 wrote:

  MM> On Sat, Aug 18, 2012 at 11:27 AM, Vitalie Spinu <spinuvit at gmail.com> wrote:
  >> >> Martin Maechler <maechler at stat.math.ethz.ch>
  >> >> on Sat, 18 Aug 2012 11:13:51 +0200 wrote:
  >> 
  >> >            ;; modified by shiba at isac 7.3.1992
  >> >            (cond ((and (numberp ess-expression-offset)
  >> >                        (re-search-backward "[ \t]*expression[ \t]*" bol t))
  >> >                   ;; This regexp match every "expression".
  >> >                   ;; modified by shiba
  >> >                   ;;(forward-sexp -1)
  >> >                   (beginning-of-line)
  >> >                   (skip-chars-forward " \t")
  >> >                   ;; End
  >> >                   (+ (current-column) ess-expression-offset))
  >> 
  >> > and it is clearly why it leads to the wrong indentation here.
  >> 
  >> But, cannot we just remove that? Why would we treat "expression"
  >> differently from other functions? There are other functions with the
  >> same semantics (quote, bquote, substitute) and we are not treating them
  >> separately.

  MM> they have a very different semantic:  expression is very much  list( ...)

Hm, then how do you explain this:

    expression(aa = 1,
        bb = 2)   
    
    list(aa = 1,
         bb = 2)
    
One more argument for removal :) At the end it's just a function
call. Why would it be so special?

  MM> Apropos: New bug report (of new behavior):

  MM> M-$ (ispell-word)  inside a simple word in  *R*  now fails
  MM> with
  MM>         Invalid function: ("^<<.*>>=" . "^@")

So is that a bug or not? I didn't really understand your comment in the
other mail.

Vitalie.



More information about the ESS-bugs mailing list