haskell-indentation-indent-region and smartparens interact poorly

[rpglover64]

Calling a smartparens command like sp-raise-sexp or sp-forward-slurp calls indent-region which calls haskell-indentation-indent-region, which indents the region exactly 1 space, which is almost never what is wanted.

I think this is a misfeature in haskell-indentation-indent-region.

[gracjan]

Region indentation can be considered unimplemented feature. We do not have that working yet.

[rpglover64]

Then it would be more useful if region indentation did absolutely nothing.

[gracjan]

@rpglover64: Are you able to submit a unit test in tests/haskell-indentation-tests.el that shows the broken behavior?

[rpglover64]

Not in the style of the file (i.e. not using haskell-indentation-check); AFAICT, haskell-indentation-find-indentations is behaving correctly, and that's what the file is testing.

I personally worked around this by redefining haskell-indentation-indent-region to do nothing.

[gracjan]

It is expected that there will be more check styles.

[gracjan]

Noted the need of indent-region in #1032.
And closing this particular issue because it seems that smartparens has a flag to work around haskell-mode missing feature.

[khanage]

What is the flag that works around this @gracjan?

[gracjan]

The flag is in smart parens.

[bergey]

@khanage From my answer to #1031:

(add-to-list 'sp-no-reindent-after-kill-modes 'haskell-mode)

That works for sp-kill-sexp. It doesn't work (in my quick test) for sp-raise-sexp or sp-forward-slurp. Maybe smartparens should apply sp-no-reindent-after-kill-modes to these functions also; I'm not certain. I think #1032 covers the issue from haskell-mode perspective.