search

haskell-suite / haskell-names

Haskell suite library for name resolution
52 stars 17 forks source link

Add extra info to each constructor, regarding its arity and field names. #38

Closed ghost closed 11 years ago

ghost commented 11 years ago

(see #37 for a similar patch; should be mutually exclusive; I prefer #37)

UnkindPartition commented 11 years ago

Hmm... Didn't we discuss that on reddit?

ghost commented 11 years ago

Yes, we did. I understand this falls just outside the scope of haskell-names. However, I thought it was such a simple patch that I'd share it.

UnkindPartition commented 11 years ago

Yes, it is indeed simple. The problem with it is that it's one of dozens of other similar changes every one of which makes sense, but together they lead to the constantly changing interface format and un-modular architecture.

E.g. someone would want to store the bang and UNPACK annotations; someone would want to store the types; etc.

To quote myself:

My philosophy is that haskell-names contains just enough information so that one can resolve names.

If it's not needed for name resolution, it doesn't belong here. I'd rather you go the proper path and create your own interface files for the kind of information you need. I am happy to help should you run into any problems.

UnkindPartition commented 11 years ago

Oh, and for the reference, here's the reddit thread: http://www.reddit.com/r/haskell/comments/1mbaw0/recordwildcards_are_evil/cc7jr8x