Wrapped signatures not highlighted #39

Closed
feuerbach opened this Issue Nov 20, 2012 · 4 comments

Comments

Projects
None yet
2 participants
@feuerbach
Contributor

feuerbach commented Nov 20, 2012

I often use this style for long type signatures:

buildLib
  :: Verbosity -> PackageDescription -> LocalBuildInfo
  -> Library -> ComponentLocalBuildInfo -> IO ()

Currently vim2hs doesn't recognize this as a signature.

@dag dag closed this in 4451e85 Nov 20, 2012

@feuerbach

This comment has been minimized.

Show comment Hide comment
@feuerbach

feuerbach Nov 20, 2012

Contributor

That highlights the type (the last 2 lines in the example), but the function name ("buildLib") is still of a different colour.

Contributor

feuerbach commented Nov 20, 2012

That highlights the type (the last 2 lines in the example), but the function name ("buildLib") is still of a different colour.

@dag dag reopened this Nov 21, 2012

@dag

This comment has been minimized.

Show comment Hide comment
@dag

dag Nov 21, 2012

Owner

The fix I come up with breaks top-level TH. I'm leaning towards disabling that TH highlighting by default; it's technically incomplete anyway (it assumes you start with an identifier, but technically any expression is valid). Thoughts?

Owner

dag commented Nov 21, 2012

The fix I come up with breaks top-level TH. I'm leaning towards disabling that TH highlighting by default; it's technically incomplete anyway (it assumes you start with an identifier, but technically any expression is valid). Thoughts?

@feuerbach

This comment has been minimized.

Show comment Hide comment
@feuerbach

feuerbach Nov 21, 2012

Contributor

Personally I won't miss that too much, as I use TH very rarely. But others may disagree.

How about adding an option to choose between these two alternatives?

Contributor

feuerbach commented Nov 21, 2012

Personally I won't miss that too much, as I use TH very rarely. But others may disagree.

How about adding an option to choose between these two alternatives?

@dag

This comment has been minimized.

Show comment Hide comment
@dag

dag Nov 21, 2012

Owner

Such an option would be hard to test and maintain properly; I'd really prefer not to go that route.

Owner

dag commented Nov 21, 2012

Such an option would be hard to test and maintain properly; I'd really prefer not to go that route.

@dag dag closed this in 3be347f Nov 26, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment