{"comment":{"url":"https://api.github.com/repos/emacs-ess/ESS/comments/11669988","html_url":"https://github.com/emacs-ess/ESS/commit/f15e64fff58c8a79101fc67559671c07d2dda7e7#commitcomment-11669988","id":11669988,"user":{"login":"lionel-","id":4465050,"avatar_url":"https://avatars.githubusercontent.com/u/4465050?v=3","gravatar_id":"","url":"https://api.github.com/users/lionel-","html_url":"https://github.com/lionel-","followers_url":"https://api.github.com/users/lionel-/followers","following_url":"https://api.github.com/users/lionel-/following{/other_user}","gists_url":"https://api.github.com/users/lionel-/gists{/gist_id}","starred_url":"https://api.github.com/users/lionel-/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lionel-/subscriptions","organizations_url":"https://api.github.com/users/lionel-/orgs","repos_url":"https://api.github.com/users/lionel-/repos","events_url":"https://api.github.com/users/lionel-/events{/privacy}","received_events_url":"https://api.github.com/users/lionel-/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"f15e64fff58c8a79101fc67559671c07d2dda7e7","created_at":"2015-06-14T13:47:38Z","updated_at":"2015-06-14T13:47:38Z","body":"@mmaechler it would be better to post an issue (mentioning my github handle) instead of changing the test files.\r\n\r\nI'm not sure this is a bug rather than a consequence of the new default for blocks, which is to indent based on the previous line. This is also how RStudio handles this. The behaviour you want would be reproduced by setting `ess-offset-block` to `open-delim`. But then you'd have lambda functions and other block arguments also indented that way.\r\n\r\nMaybe we should add some overrides for if-else blocks and function declarations?\r\n```\r\n(ess-indent-at-open-paren  .   (if-else decl-block))\r\n```\r\n\r\nI'm starting to wonder if we should change the interface again and have three settings `ess-indent-at-open-delim`, `ess-indent-at-prev-line`, `ess-indent-at-prev-call` that could take a list containing a subset of `(arg arg-newline block decl-block decl-arg if-else)`. That would make things more modular.\r\n\r\n@vspinu what do you think?"}}