<div dir="ltr"><div>At the moment in HaRe we do use the wrapped location, but it results in code that is much more complex. This change is more about bringing predictability to the AST than anything else.<br><br></div>If anyone feels strongly that it should not happen then we can live without it.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 26, 2015 at 2:11 PM, GHC <span dir="ltr"><<a href="mailto:ghc-devs@haskell.org" target="_blank">ghc-devs@haskell.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">#11019: ApiAnnotations: Make all RdrName occurences Located<br>
-------------------------------------+-------------------------------------<br>
        Reporter:  alanz             |                Owner:  alanz<br>
            Type:  bug               |               Status:  new<br>
        Priority:  normal            |            Milestone:  8.0.1<br>
       Component:  Compiler          |              Version:  7.10.2<br>
</span>      Resolution:                    |             Keywords:<br>
                                     |  ApiAnnotations<br>
Operating System:  Unknown/Multiple  |         Architecture:<br>
                                     |  Unknown/Multiple<br>
 Type of failure:  None/Unknown      |            Test Case:<br>
<span class="">      Blocked By:                    |             Blocking:<br>
 Related Tickets:                    |  Differential Rev(s):<br>
       Wiki Page:                    |<br>
-------------------------------------+-------------------------------------<br>
<br>
</span>Comment (by goldfire):<br>
<br>
 Perhaps I'm missing something very simple, but if you have a, for example,<br>
 `LHsExpr Name` which matches the pattern `(L loc (HsVar name))`, the `loc`<br>
 should apply just as well to the `name` as to the overall expression. My<br>
 understanding of each of the cases above is that they all have this<br>
 property, so adding another `Located` wrapper would be redundant.<br>
<br>
 Or am I missing something?<br>
<br>
--<br>
Ticket URL: <<a href="http://ghc.haskell.org/trac/ghc/ticket/11019#comment:1" rel="noreferrer" target="_blank">http://ghc.haskell.org/trac/ghc/ticket/11019#comment:1</a>><br>
<div class="HOEnZb"><div class="h5">GHC <<a href="http://www.haskell.org/ghc/" rel="noreferrer" target="_blank">http://www.haskell.org/ghc/</a>><br>
The Glasgow Haskell Compiler<br>
</div></div></blockquote></div><br></div>