
#8607: Invalid location reported for type constructors -------------------------------------+------------------------------------ Reporter: edsko | Owner: goldfire Type: bug | Status: new Priority: normal | Milestone: Component: Compiler | Version: 7.7 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: Unknown/Multiple Type of failure: None/Unknown | Difficulty: Unknown Test Case: | Blocked By: Blocking: | Related Tickets: -------------------------------------+------------------------------------ Comment (by edsko): Well I'm not familiar at all with this code, so I may be talking nonsense; I just wanted to point out that a `Located Name` is _not_ redundant: a `Located Name` contains two locations, and that's exactly right: one if the location of the identifier, and the other is the def span of whatever the identifier is referring to. The change in this commit changed _both_ of those locations to be the span of the entire type declaration; it seems to be that it ought to be sufficient to change only the `nameSrcSpan` but not the location of the identifier itself. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/8607#comment:9 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler